zabetak opened a new pull request, #5919: URL: https://github.com/apache/hive/pull/5919
### What changes were proposed in this pull request? 1. Allow to define name/id for each database used via the qt:database directive 2. Expose database connection properties via system properties and manage them during the life-cycle of the database ### Why are the changes needed? 1. Facilitate writing of qtests for the JDBC storage handler 3. Avoid harcoding connection properties in qfiles 4. Opens up the stage for supporting multiple databases of the same kind in qtests ### Does this PR introduce _any_ user-facing change? No, it only affects the test infrastructure. ### How was this patch tested? ``` mvn test -Dtest=TestMiniLlapLocalCliDriver -Dqfile=cbo_jdbc_joincost.q,dataconnector_mysql.q,jdbc_filter_expand_row_operator.q,jdbc_partition_table_pruned_pcolumn.q,jdbc_project_pushdown.q,jdbc_table_dml_postgres.q,jdbc_table_limit_postgres.q,jdbc_table_with_schema_mariadb.q,jdbc_table_with_schema_mssql.q,jdbc_table_with_schema_oracle.q,jdbc_table_with_schema_postgres.q,qt_database_mssql.q,qt_database_mysql.q,qt_database_postgres.q -Dtest.output.overwrite ``` -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
