Skip to content

Commit

Permalink
[#2508] improvement: Remove untracked directories when executing grad…
Browse files Browse the repository at this point in the history
…le clean (#2531)

<!--
1. Title: [#<issue>] <type>(<scope>): <subject>
   Examples:
     - "[#123] feat(operator): support xxx"
     - "[#233] fix: check null before access result in xxx"
     - "[MINOR] refactor: fix typo in variable name"
     - "[MINOR] docs: fix typo in README"
     - "[#255] test: fix flaky test NameOfTheTest"
   Reference: https://www.conventionalcommits.org/en/v1.0.0/
2. If the PR is unfinished, please mark this PR as draft.
-->

### What changes were proposed in this pull request?

Remove directory `spark-warehouse` when exectue gradle clean in moudle
catalog-lakehouse-iceberg.
Remove directory `src/main/resources` when execute gradle clean in
module server, since which is generated by gradle build.

### Why are the changes needed?

Fix: #2508

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

`./gradlew :catalogs:catalog-lakehouse-iceberg:clean`
`./gradlew :server:clean`

---------

Co-authored-by: Qi Yu <[email protected]>
  • Loading branch information
yijhenlin and yuqi1129 authored Mar 14, 2024
1 parent 94912a7 commit e65ba06
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -48,4 +48,5 @@ include web/.gitignore
.node

# integration test
integration-test/metastore_db
integration-test/metastore_db
catalogs/catalog-lakehouse-iceberg/spark-warehouse
4 changes: 4 additions & 0 deletions catalogs/catalog-lakehouse-iceberg/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -168,3 +168,7 @@ tasks.test {
init(this)
}
}

tasks.clean {
delete("spark-warehouse")
}
2 changes: 1 addition & 1 deletion server/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,6 @@ tasks {
environment("GRAVITINO_TEST", "true")
}
clean {
delete("$propertiesFile")
delete(file(propertiesFile).getParentFile())
}
}

0 comments on commit e65ba06

Please sign in to comment.