jcamachor commented on a change in pull request #1482:
URL: https://github.com/apache/hive/pull/1482#discussion_r487299089
##########
File path: ql/src/test/queries/clientpositive/external_jdbc_table2.q
##########
@@ -7,43 +6,43 @@ CREATE TEMPORARY FUNCTION dboutput AS
'org.apache.hadoop.hive.contrib.genericudf
FROM src
SELECT
-dboutput
('jdbc:derby:;databaseName=${system:test.tmp.dir}/test_derby_auth1;create=true','user1','passwd1',
+dboutput
('jdbc:derby:;databaseName=${system:test.tmp.dir}/test_derby_auth1_1;create=true','user1','passwd1',
Review comment:
I changed it to avoid a clash with other test in the temp directory,
which I believe was causing HIVE-23910.
##########
File path:
ql/src/test/results/clientpositive/llap/external_jdbc_table_perf.q.out
##########
@@ -6115,32 +6112,39 @@ WHERE "d_year" = 1999 AND "d_moy" BETWEEN 1 AND 3 AND
"d_date_sk" IS NOT NULL) A
Reduce Operator Tree:
Merge Join Operator
condition map:
- Anti Join 0 to 1
+ Left Outer Join 0 to 1
Review comment:
This change is not actually related to this patch. Note that the test
was disabled by default (HIVE-23910); it seems maybe a preliminary version of
HIVE-23716 changed these q files and it should have not.
----------------------------------------------------------------
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
