----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66774/#review201826 -----------------------------------------------------------
Thanks Marta for the patch! These messages will be usefull. Two small things / questions below. Peter service/src/java/org/apache/hive/service/cli/operation/GetCatalogsOperation.java Lines 66 (patched) <https://reviews.apache.org/r/66774/#comment283421> Does this result a duplication in the log for the exception messages? service/src/java/org/apache/hive/service/cli/operation/GetPrimaryKeysOperation.java Lines 113 (patched) <https://reviews.apache.org/r/66774/#comment283422> nit: formatting? - Peter Vary On April 24, 2018, 10:34 a.m., Marta Kuczora wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66774/ > ----------------------------------------------------------- > > (Updated April 24, 2018, 10:34 a.m.) > > > Review request for hive and Peter Vary. > > > Bugs: HIVE-19285 > https://issues.apache.org/jira/browse/HIVE-19285 > > > Repository: hive-git > > > Description > ------- > > Subclasses of MetaDataOperation are not writing anything to the logs. It > would be useful to have some INFO and DEBUG level logging in these classes. > > > Diffs > ----- > > > service/src/java/org/apache/hive/service/cli/operation/GetCatalogsOperation.java > 7944467 > > service/src/java/org/apache/hive/service/cli/operation/GetColumnsOperation.java > d67ea90 > > service/src/java/org/apache/hive/service/cli/operation/GetCrossReferenceOperation.java > 99ccd4e > > service/src/java/org/apache/hive/service/cli/operation/GetFunctionsOperation.java > 091bf50 > > service/src/java/org/apache/hive/service/cli/operation/GetPrimaryKeysOperation.java > e603fdd > > service/src/java/org/apache/hive/service/cli/operation/GetSchemasOperation.java > de09ec9 > > service/src/java/org/apache/hive/service/cli/operation/GetTableTypesOperation.java > 59cfbb2 > > service/src/java/org/apache/hive/service/cli/operation/GetTablesOperation.java > c9233d0 > > service/src/java/org/apache/hive/service/cli/operation/GetTypeInfoOperation.java > ac078b4 > > service/src/java/org/apache/hive/service/cli/operation/MetadataOperation.java > bf7c021 > > > Diff: https://reviews.apache.org/r/66774/diff/1/ > > > Testing > ------- > > Just adding some additional log messages. Tested locally by checking the log > messages for different use cases > > > Thanks, > > Marta Kuczora > >