Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/20420 )
Change subject: IMPALA-12231: Bump GBN to get HMS thrift API changes ...................................................................... Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/20420/8/tests/custom_cluster/test_events_custom_configs.py File tests/custom_cluster/test_events_custom_configs.py: http://gerrit.cloudera.org:8080/#/c/20420/8/tests/custom_cluster/test_events_custom_configs.py@522 PS8, Line 522: self.client.execute("refresh {}.{}".format(unique_database, test_table)) : self.run_stmt_in_hive( : "insert into {}.{} partition(year=2023) values (4),(5),(6)" : .format(unique_database, test_table)) : data = FireEventRequestData() : if is_insert: : insert_data = InsertEventRequestData() : insert_data.filesAdded = "/warehouse/mytable/b1" : insert_data.replace = False : data.insertData = insert_data : else: : data.refreshEvent = True : req = FireEventRequest(True, data) : req.dbName = unique_database : req.tableName = test_table : req.partitionVals = ["2023"] : self.hive_client.fire_listener_event(req) : self.run_stmt_in_hive( : "drop table {}.{}".format(unique_database, test_table)) Too many statements here to finish in 5s. This test always passes in my env without the fix. Let's bump hms_event_polling_interval_s to 10 so it can always fails before we merge this fix. -- To view, visit http://gerrit.cloudera.org:8080/20420 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I117873b628aed3e24280f9fcd79643f918c8d5f3 Gerrit-Change-Number: 20420 Gerrit-PatchSet: 8 Gerrit-Owner: Sai Hemanth Gantasala <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]> Gerrit-Comment-Date: Fri, 08 Sep 2023 00:31:18 +0000 Gerrit-HasComments: Yes
