Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/21401 )
Change subject: IMPALA-13061: Set transactional=false on query live table ...................................................................... Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/21401/3/tests/custom_cluster/test_query_live.py File tests/custom_cluster/test_query_live.py: http://gerrit.cloudera.org:8080/#/c/21401/3/tests/custom_cluster/test_query_live.py@124 PS3, Line 124: describe_ext_result = self.execute_query('describe extended sys.impala_query_live') : assert len(describe_ext_result.data) == 82 Can you add assertion for other table properties of impala_query_live here? http://gerrit.cloudera.org:8080/#/c/21401/3/tests/custom_cluster/test_query_live.py@175 PS3, Line 175: esult = self.client.execute("select * from functional.alltypes", : fetch_profile_after_close=True) : Can you also assert that 'transactional'='false' in table properties via "DESCRIBE EXTENDED" query? -- To view, visit http://gerrit.cloudera.org:8080/21401 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie60a2bd03fabc63c85bcd9fa2489e9d47cd2aa65 Gerrit-Change-Number: 21401 Gerrit-PatchSet: 3 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Andrew Sherman <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Tue, 07 May 2024 16:51:38 +0000 Gerrit-HasComments: Yes
