-
Notifications
You must be signed in to change notification settings - Fork 416
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[#5289][Improvement] CatalogMysqlDriverIT miss driver url #6671
base: main
Are you sure you want to change the base?
Conversation
I used jdk17 locally for integration testing and got an error, but I saw that the pg database image was down and failed to start. Can you help me check it out? ./gradlew test -PskipTests -PtestMode=deploy -PjdkVersion=17 -PjdbcBackend=postgresql -PskipDockerTests=false -x :web:web:test -x :web:integration-test:test -x :clients:client-python:test -x :flink-connector:flink:test -x :spark-connector:spark-common:test -x :spark-connector:spark-3.3:test -x :spark-connector:spark-3.4:test -x :spark-connector:spark-3.5:test -x :spark-connector:spark-runtime-3.3:test -x :spark-connector:spark-runtime-3.4:test -x :spark-connector:spark-runtime-3.5:test -x :trino-connector:integration-test:test -x :trino-connector:trino-connector:test -x :authorizations:authorization-chain:test -x :authorizations:authorization-ranger:test
HadoopCatalogIT > initializationError FAILED
HadoopS3CatalogIT > initializationError FAILED
HadoopUserAuthenticationIT > initializationError FAILED HadoopUserImpersonationIT > initializationError FAILED 59 tests completed, 4 failed, 54 skipped
FAILURE: Build failed with an exception.
Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0. You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins. For more on this, please refer to https://docs.gradle.org/8.2/userguide/command_line_interface.html#sec:command_line_warnings in the Gradle documentation. BUILD FAILED in 2m 57s |
@lsyulong |
What changes were proposed in this pull request?
#5289 [Improvement] CatalogMysqlDriverIT miss driver url
Why are the changes needed?
#5289 [Improvement] CatalogMysqlDriverIT miss driver url
because,I have submitted a PR before, but there were some problems. Later, I built the environment locally and revised it again.
Fix: #5289
Does this PR introduce any user-facing change?
no
How was this patch tested?
yes ,I have successfully built and tested locally