[ https://issues.apache.org/jira/browse/HIVE-16643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16021077#comment-16021077 ]
Yongzhi Chen commented on HIVE-16643: ------------------------------------- The patch looks good +1 > BeeLine tests output should keep the PREHOOK/POSTHOOK Input/Output orderdering > ------------------------------------------------------------------------------ > > Key: HIVE-16643 > URL: https://issues.apache.org/jira/browse/HIVE-16643 > Project: Hive > Issue Type: New Feature > Components: Testing Infrastructure > Reporter: Peter Vary > Assignee: Peter Vary > Attachments: HIVE-16643.01.patch, HIVE-16643.patch > > > The {{PreExecutePrinter}} and the {{PostExecutePrinter}} prints the query > input and the output list in alphabetical order in {{printEntities}} method. > Our goal is to have the same output from the BeeLine query tests, and the Cli > query tests. Since the BeeLine tests are using test specific databases to run > the tests, and only converting the results in the end to remove this specific > database names from the output, we have to reorder the lists after this > conversion. > Raw BeeLine output: > {code} > [..] > INFO : PREHOOK: Output: create_merge_compressed@src_rc_merge_test > INFO : PREHOOK: Output: database:create_merge_compressed > [..] > {code} > Before patch BeeLine output: > {code} > [..] > PREHOOK: Output: default@src_rc_merge_test > PREHOOK: Output: database:default > [..] > {code} > Expected output: > {code} > [..] > PREHOOK: Output: database:default > PREHOOK: Output: default@src_rc_merge_test > [..] > {code} -- This message was sent by Atlassian JIRA (v6.3.15#6346)