hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Peter Vary <>
Subject Re: Review Request 59096: HIVE-16607 ColumnStatsAutoGatherContext regenerates HiveConf.HIVEQUERYID
Date Wed, 10 May 2017 09:18:00 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated May 10, 2017, 9:17 a.m.)

Review request for hive, Aihua Xu and pengcheng xiong.


Addressed review comments:
- Renamed QueryStateBuilder to Builder
- Renamed setter methods to "with" methods
- Added withHiveConf, and removed build()
- When possible moved from "conf.getVar(HiveConf.ConfVars.HIVEQUERYID)" to "queryState.getQueryID"

Removed confOverlay local variable from the Operation class - using queryState instead

Bugs: HIVE-16607

Repository: hive-git


When creating a QueryState object the caller could specify if new QueryID should be created
or the exisiting should be used.
Created a QueryStateBuilder to make the QueryState object creation more readable.
New QueryId is only created in two places:
- Driver constructor
- Operation constructor
Otherwise the existing queryId is used

Diffs (updated)

  itests/src/test/resources/ 5ab3076 
  itests/util/src/main/java/org/apache/hadoop/hive/ql/ b897ffa 
  itests/util/src/main/java/org/apache/hive/beeline/ 3d9ca99 
  itests/util/src/main/java/org/apache/hive/beeline/ 7c50e18 
  ql/src/java/org/apache/hadoop/hive/ql/ 29cce9a 
  ql/src/java/org/apache/hadoop/hive/ql/ 6dfaa9f 
  ql/src/java/org/apache/hadoop/hive/ql/exec/ cf575de 
  ql/src/java/org/apache/hadoop/hive/ql/io/rcfile/stats/ 77bce97 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/ b121eea

  ql/src/java/org/apache/hadoop/hive/ql/parse/ 3b719af 
  ql/src/test/org/apache/hadoop/hive/ql/exec/ c7266bc 
  ql/src/test/org/apache/hadoop/hive/ql/parse/ c734988 
  ql/src/test/org/apache/hadoop/hive/ql/parse/ 201622e 
  ql/src/test/org/apache/hadoop/hive/ql/parse/ e607f10 
  ql/src/test/org/apache/hadoop/hive/ql/parse/ 2674835 
  ql/src/test/org/apache/hadoop/hive/ql/parse/ 80865bd

  ql/src/test/org/apache/hadoop/hive/ql/parse/ 5849950 
  ql/src/test/org/apache/hadoop/hive/ql/parse/ a573808

  ql/src/test/org/apache/hadoop/hive/ql/parse/authorization/ 5d01080

  ql/src/test/org/apache/hadoop/hive/ql/parse/authorization/ c552ba7

  ql/src/test/results/clientpositive/beeline/materialized_view_create_rewrite.q.out PRE-CREATION

  service/src/java/org/apache/hive/service/cli/operation/ 0b27608 
  service/src/java/org/apache/hive/service/cli/operation/ 0b51591 




Added new BeeLine test - The original code made the test output different from the Cli test
output, since the QueryLog was truncated when the queryId was changed. After the change the
BeeLine test output is exactly the same as the Cli output.


Peter Vary

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message