----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75244/#review226999 -----------------------------------------------------------
hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveClient.java Line 829 (original), 829 (patched) <https://reviews.apache.org/r/75244/#comment315274> HiveClient is used to lookup resources in Hive to provide auto-completion in Ranger policy UI. This can be used to connect to earlier version of Hive as well. Has this change been tested to connect with earlier version of Hive as well? - Madhan Neethiraj On Oct. 24, 2024, 2:26 p.m., Himanshu Maurya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/75244/ > ----------------------------------------------------------- > > (Updated Oct. 24, 2024, 2:26 p.m.) > > > Review request for ranger, bhavik patel, Dhaval Shah, Dineshkumar Yadav, > Harshal Chavan, Kishor Gollapalliwar, Madhan Neethiraj, Mehul Parikh, Nitin > Galave, Pradeep Agrawal, and Velmurugan Periasamy. > > > Bugs: RANGER-4968 > https://issues.apache.org/jira/browse/RANGER-4968 > > > Repository: ranger > > > Description > ------- > > Service Test Connection for Hive service is not working in ranger post adding > support for Hive4. > Because of the cmd - show databases like "*"; used while service check, is no > longer working in Hive4. As per > [HIVE-23243](https://issues.apache.org/jira/browse/HIVE-23243) , the pattern > * is replaced with %. So in ranger also we will have to use % instead of *. > > > Diffs > ----- > > > hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveClient.java > 6d6bbbb69 > > > Diff: https://reviews.apache.org/r/75244/diff/1/ > > > Testing > ------- > > As part of testing I have > 1. Installed a cluster having Ranger and Hive4 > 2. From ranger admin UI now the service test connection for Hive service is > successful > > > Thanks, > > Himanshu Maurya > >
