----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73424/#review223153 -----------------------------------------------------------
plugin-presto/src/main/java/org/apache/ranger/services/presto/client/PrestoResourceManager.java Line 44 (original), 44 (patched) <https://reviews.apache.org/r/73424/#comment312259> Error message need not to have "==>". Symbol "==>/<==" is to show in debug log entry/exit of the routine. please consider correcting it all off this occurance. plugin-presto/src/main/java/org/apache/ranger/services/presto/client/PrestoResourceManager.java Line 109 (original), 109 (patched) <https://reviews.apache.org/r/73424/#comment312258> <== instead of ==> - Ramesh Mani On June 15, 2021, 9:46 p.m., Naoki Takezoe wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73424/ > ----------------------------------------------------------- > > (Updated June 15, 2021, 9:46 p.m.) > > > Review request for ranger. > > > Repository: ranger > > > Description > ------- > > Would make logging in Presto plugin more consistent. > > > Diffs > ----- > > > plugin-presto/src/main/java/org/apache/ranger/services/presto/RangerServicePresto.java > d95876a7d > > plugin-presto/src/main/java/org/apache/ranger/services/presto/client/PrestoClient.java > 7b55b77bd > > plugin-presto/src/main/java/org/apache/ranger/services/presto/client/PrestoResourceManager.java > 008bf0fa6 > > > Diff: https://reviews.apache.org/r/73424/diff/1/ > > > Testing > ------- > > > Thanks, > > Naoki Takezoe > >
