----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40549/#review107399 -----------------------------------------------------------
LGTM. Couple of questions. service/src/java/org/apache/hive/service/cli/operation/Operation.java (line 56) <https://reviews.apache.org/r/40549/#comment166493> what does MDC stand for ? service/src/java/org/apache/hive/service/cli/operation/Operation.java (lines 56 - 57) <https://reviews.apache.org/r/40549/#comment166496> why make these public ... only accessed in this class. service/src/java/org/apache/hive/service/cli/operation/Operation.java (line 87) <https://reviews.apache.org/r/40549/#comment166497> is this conditional needed ? - Mohit Sabharwal On Nov. 20, 2015, 4:56 p.m., Aihua Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40549/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2015, 4:56 p.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > HIVE-12456: QueryId can't be stored in the configuration of the SessionState > since multiple queries can run in a single session > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java > 4f8209ad90841740c447534f7902d49c8a8d2038 > > service/src/java/org/apache/hive/service/cli/operation/ExecuteStatementOperation.java > 3f2de108f069a815a46d49498238bb8197263d93 > service/src/java/org/apache/hive/service/cli/operation/Operation.java > d13415e820e67a027d63ba994c05db18b613afc1 > service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java > 8b4226522f1ff6c5c777fe372490f0fd157a3b2b > service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java > 2d784f0d978c04ce674dc06b1adc0cf3d8d5f984 > > Diff: https://reviews.apache.org/r/40549/diff/ > > > Testing > ------- > > > Thanks, > > Aihua Xu > >
