nevzheng commented on code in PR #13517:
URL: https://github.com/apache/gravitino/pull/13517#discussion_r4120237008
##########
.github/workflows/build.yml:
##########
@@ -147,24 +151,26 @@ jobs:
spark-connector/**/*.log
build:
- # The type of runner that the job will run on
+ name: build (${{ matrix.java-version }}, ${{ matrix.shard }})
runs-on: ubuntu-latest
strategy:
+ fail-fast: false
matrix:
java-version: [ 17 ]
- timeout-minutes: 120
+ # Shards are defined in dev/ci/test-shards.sh.
+ shard: ${{ fromJSON(needs.changes.outputs.build_shards) }}
+ timeout-minutes: 90
Review Comment:
Done — reverted to 120 min in 654cfe33a.
##########
core/build.gradle.kts:
##########
@@ -103,6 +106,107 @@ artifacts {
add("testArtifacts", testJar)
}
+val coreDatabaseTestTag = "gravitino-core-database-test"
+val coreH2TestTag = "gravitino-core-h2-test"
+val coreMySQLTestTag = "gravitino-core-mysql-test"
+val corePostgreSQLTestTag = "gravitino-core-postgresql-test"
+val coreTestBackendProperty = "gravitino.core.test.backend"
+
+fun registerCoreTestTask(
+ taskName: String,
+ backend: String? = null
+) = tasks.register<Test>(taskName) {
+ group = "verification"
+ description =
+ if (backend == null) {
+ "Runs core unit tests."
+ } else {
+ "Runs core database tests against $backend."
+ }
+
+ testClassesDirs = sourceSets["test"].output.classesDirs
+ classpath = sourceSets["test"].runtimeClasspath
+
+ inputs.property("coreTestSuite", backend ?: "unit")
+ inputs.property("coreTestBackend", backend ?: "none")
+ inputs.property("includeDockerTaggedTests", backend != null)
Review Comment:
Done — renamed to coreTestIncludesDockerTaggedTests in 654cfe33a.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
