[ https://issues.apache.org/jira/browse/IMPALA-9311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17020716#comment-17020716 ]
Anurag Mantripragada edited comment on IMPALA-9311 at 1/22/20 2:04 AM: ----------------------------------------------------------------------- Thanks for creating the JIRA. In my local tests, hms always returned the primary keys in the reverse order of their definition. Obviously there is some inconsistency here. I will dig deeper into the HMS code to see if there is a specific pattern. If not, this could make a bunch of catalog tests that I wrote for PK/FK also flaky. was (Author: anuragmantri): Thanks for creating the JIRA. In my local tests, hms always returned the primary keys in the reverse order of their definition. Obviously there is some inconsistency here. I will dig deeper into the HMS code to see if there is a specific pattern if not, this could make a bunch of catalog tests that I wrote for PK/FK also flaky. > test_show_create_table failed with primary key mismatch > ------------------------------------------------------- > > Key: IMPALA-9311 > URL: https://issues.apache.org/jira/browse/IMPALA-9311 > Project: IMPALA > Issue Type: Test > Affects Versions: Impala 3.4.0 > Reporter: Xiaomeng Zhang > Assignee: Anurag Mantripragada > Priority: Major > Labels: broken-build > > {code:java} > Error Messagemetadata/test_show_create_table.py:62: in test_show_create_table > unique_database) metadata/test_show_create_table.py:110: in > __run_show_create_table_test_case self.__compare_result(expected_result, > create_table_result) metadata/test_show_create_table.py:146: in > __compare_result assert expected_sql_filtered == actual_sql_filtered E > assert "CREATE EXTER...parent_table'" == "CREATE EXTERN...parent_table'" E > Skipping 71 identical leading characters in diff, use -v to show E > Skipping 126 identical trailing characters in diff, use -v to show E - > MARY KEY (year, id)) ROW FO E ? ---- E + MARY KEY (id, > year)) ROW FO E ? > ++++Stacktracemetadata/test_show_create_table.py:62: in test_show_create_table > unique_database) > metadata/test_show_create_table.py:110: in __run_show_create_table_test_case > self.__compare_result(expected_result, create_table_result) > metadata/test_show_create_table.py:146: in __compare_result > assert expected_sql_filtered == actual_sql_filtered > E assert "CREATE EXTER...parent_table'" == "CREATE EXTERN...parent_table'" > E Skipping 71 identical leading characters in diff, use -v to show > E Skipping 126 identical trailing characters in diff, use -v to show > E - MARY KEY (year, id)) ROW FO > E ? ---- > E + MARY KEY (id, year)) ROW FO > E ? ++++{code} > I think this is due to commit > [https://github.com/apache/impala/commit/cfe60858da110cf1256bd3aa5d4f8d374578a33d] -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-all-unsubscr...@impala.apache.org For additional commands, e-mail: issues-all-h...@impala.apache.org