-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70372/
-----------------------------------------------------------
(Updated April 4, 2019, 6:18 p.m.)
Review request for hive, Ashutosh Chauhan and Jason Dere.
Changes
-------
review comment update
Bugs: HIVE-21427
https://issues.apache.org/jira/browse/HIVE-21427
Repository: hive-git
Description
-------
HIVE-21427: Syslog storage handler
Diffs (updated)
-----
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java aa27a01174
data/files/syslog-hs2-2.log PRE-CREATION
data/files/syslog-hs2.log PRE-CREATION
itests/src/test/resources/testconfiguration.properties 96dfbc4b56
llap-server/src/java/org/apache/hadoop/hive/llap/io/api/impl/LlapRecordReader.java
9ef7af4eb0
ql/src/java/org/apache/hadoop/hive/ql/io/sarg/ConvertAstToSearchArg.java
27fe828b75
ql/src/java/org/apache/hadoop/hive/ql/log/syslog/SyslogInputFormat.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/log/syslog/SyslogParser.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/log/syslog/SyslogSerDe.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/log/syslog/SyslogStorageHandler.java
PRE-CREATION
ql/src/test/org/apache/hadoop/hive/ql/log/TestSyslogInputFormat.java
PRE-CREATION
ql/src/test/queries/clientpositive/syslog_parser.q PRE-CREATION
ql/src/test/queries/clientpositive/syslog_parser_file_pruning.q PRE-CREATION
ql/src/test/results/clientpositive/llap/syslog_parser.q.out PRE-CREATION
ql/src/test/results/clientpositive/llap/syslog_parser_file_pruning.q.out
PRE-CREATION
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/tools/schematool/MetastoreSchemaTool.java
eafe0c6d46
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/tools/schematool/SchemaToolCommandLine.java
6282078411
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/tools/schematool/SchemaToolTaskCreateLogsTable.java
PRE-CREATION
Diff: https://reviews.apache.org/r/70372/diff/2/
Changes: https://reviews.apache.org/r/70372/diff/1-2/
Testing
-------
Thanks,
Prasanth_J