----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59325/ -----------------------------------------------------------
(Updated May 17, 2017, 9:56 p.m.) Review request for hive, Jason Dere and Sergey Shelukhin. Changes ------- Fixes review comments. Adds one additional change to use a consistent "queryId". Earlier, RecordReader would pick up a different queryId from the rest of the system, which would geneate 2 log files. Only one of the two log files would get rotated. Bugs: HIVE-14052 https://issues.apache.org/jira/browse/HIVE-14052 Repository: hive-git Description ------- Cleanup of structures required when LLAP access from external clients completes Diffs (updated) ----- llap-common/src/gen/protobuf/gen-java/org/apache/hadoop/hive/llap/daemon/rpc/LlapDaemonProtocolProtos.java ece31ed llap-common/src/protobuf/LlapDaemonProtocol.proto 3a3a2b8 llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/QueryInfo.java ce2f457 llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/QueryTracker.java daeb555 llap-server/src/test/org/apache/hadoop/hive/llap/daemon/impl/TaskExecutorTestHelpers.java 27c426c ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDTFGetSplits.java 868eec7 Diff: https://reviews.apache.org/r/59325/diff/2/ Changes: https://reviews.apache.org/r/59325/diff/1-2/ Testing ------- Thanks, Siddharth Seth