Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22853 )

Change subject: IMPALA-14028: Refactor cancel_query_and_validate_state with HS2
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22853/4/tests/common/impala_connection.py
File tests/common/impala_connection.py:

http://gerrit.cloudera.org:8080/#/c/22853/4/tests/common/impala_connection.py@1132
PS4, Line 1132:
              :   def set_configuration_option(self, name, value, 
is_log_sql=True):
              :     # Only set the option if it's not already set to the same 
value.
              :     # value must be parsed to string.
              :     name = name.lower()
              :     value = str(value)
              :     if self.__query_options.get(name) != value:
              :       self.__query_options[name] = value
              :       if is_log_sql:
              :         self.log_client("\n\nset {0}={1};\n".format(name, 
value))
              :       return True
              :     return False
This will require some adjustment to follow https://gerrit.cloudera.org/c/22862/



--
To view, visit http://gerrit.cloudera.org:8080/22853
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I626a1a06eb3d5dc9737c7d4289720e1f52d2a984
Gerrit-Change-Number: 22853
Gerrit-PatchSet: 4
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Xuebin Su <[email protected]>
Gerrit-Comment-Date: Wed, 07 May 2025 01:57:29 +0000
Gerrit-HasComments: Yes

Reply via email to