ashniku commented on code in PR #6412:
URL: https://github.com/apache/hive/pull/6412#discussion_r3045975837


##########
jdbc/src/java/org/apache/hive/jdbc/HiveStatement.java:
##########
@@ -398,6 +407,70 @@ private TGetOperationStatusResp waitForResultSetStatus() 
throws SQLException {
     return statusResp;
   }
 
+  /**
+   * When {@code SET hive.query.timeout.seconds=...} succeeds, remember the 
effective value on the
+   * connection so {@code TIMEDOUT_STATE} can report it if the server omits 
{@code errorMessage}
+   * (HIVE-28265).
+   */
+  private void trackSessionQueryTimeoutIfSet(String sql) {
+    if (sql == null) {
+      return;
+    }
+    Matcher m = SET_HIVE_QUERY_TIMEOUT_SECONDS.matcher(sql);
+    Long lastSec = null;
+    while (m.find()) {

Review Comment:
   thank you @InvisibleProgrammer This was reproducable.
   
   Could you please let me know any setting or runtime configs, if you feel can 
fix the issue?
   
   Could you also please let me know, how to refactor? As I am new to the 
community
   
       I'm sad that I have no time to debug it out but for me, it looks 
suspicious.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to