[ 
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

Reply via email to