Quanlong Huang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23528 )

Change subject: IMPALA-12401: Support more info types for HS2 GetInfo() API
......................................................................


Patch Set 3:

> In general I think that these infos should be only filled when you are sure 
> about there value. Some of them should be not filled by the server, as they 
> provide info about the ODBC driver.

+1. Initially I just plan to add what Hive has and are missing in Impala: 
https://github.com/apache/hive/blob/4903585a34ae44bb3fec4207b5acab63f6bfc8c1/service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java#L501-L508

Adding more fields is good but we should be certain on the effects.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ce5f2b9dcc2e4633b4679b002f57b5b4ea3e8bf
Gerrit-Change-Number: 23528
Gerrit-PatchSet: 3
Gerrit-Owner: Arnab Karmakar <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Comment-Date: Tue, 14 Oct 2025 08:15:31 +0000
Gerrit-HasComments: No

Reply via email to