CycloneBoy opened a new pull request #3194: URL: https://github.com/apache/incubator-dolphinscheduler/pull/3194
fix the bug #2923 ## *Tips* - *Thanks very much for contributing to Apache DolphinScheduler.* - *Please review https://dolphinscheduler.apache.org/en-us/community/index.html before opening a pull request.* ## What is the purpose of the pull request *This pull request fix the bug #2923* Corrected the connection string of hive jdbc link string. If the parameter in the link string is the configuration parameter in hive-default.xml, add it before the parameter ?. <br> > see the hive Connection URLs - Connection URL Format [HiveServer2Clients-ConnectionURLs](https://cwiki.apache.org/confluence/display/Hive/HiveServer2+Clients#HiveServer2Clients-ConnectionURLs) ``` Connection URLs Connection URL Format The HiveServer2 URL is a string with the following syntax: jdbc:hive2://<host1>:<port1>,<host2>:<port2>/dbName;initFile=<file>;sess_var_list?hive_conf_list#hive_var_list where <host1>:<port1>,<host2>:<port2> is a server instance or a comma separated list of server instances to connect to (if dynamic service discovery is enabled). If empty, the embedded server will be used. dbName is the name of the initial database. <file> is the path of init script file (Hive 2.2.0 and later). This script file is written with SQL statements which will be executed automatically after connection. This option can be empty. sess_var_list is a semicolon separated list of key=value pairs of session variables (e.g., user=foo;password=bar). hive_conf_list is a semicolon separated list of key=value pairs of Hive configuration variables for this session hive_var_list is a semicolon separated list of key=value pairs of Hive variables for this session. Special characters in sess_var_list, hive_conf_list, hive_var_list parameter values should be encoded with URL encoding if needed. ``` ## Brief change log - **Ｍodify the function of org.apache.dolphinscheduler.dao.datasource.HiveDataSource#filterOther** - **Add HiveDataSourceTest.java for the test case of the HiveDataSource#filterOther** ## Verify this pull request This change added tests and can be verified as follows: - *Added HiveDataSourceTest to verify the change.* ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
