----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34713/#review85793 -----------------------------------------------------------
ql/src/test/queries/clientpositive/insert_into1.q <https://reviews.apache.org/r/34713/#comment137571> ws issue ql/src/test/results/clientpositive/stats12.q.out <https://reviews.apache.org/r/34713/#comment137578> that's not correct, is it? why are the column stats true now? and there's more than 0 rows in the partition. ql/src/test/results/clientpositive/stats5.q.out <https://reviews.apache.org/r/34713/#comment137590> analyze_src is copy of src.. so num rows 1 and complete stats is incorrect no? - Gunther Hagleitner On May 28, 2015, 7:25 a.m., Ashutosh Chauhan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34713/ > ----------------------------------------------------------- > > (Updated May 28, 2015, 7:25 a.m.) > > > Review request for hive and Gopal V. > > > Bugs: HIVE-10807 > https://issues.apache.org/jira/browse/HIVE-10807 > > > Repository: hive-git > > > Description > ------- > > Invalidate basic stats for insert queries if autogather=false > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/QueryProperties.java e8f7fba > ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRFileSink1.java e5b9c2b > ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java acd9bf5 > ql/src/java/org/apache/hadoop/hive/ql/parse/QBParseInfo.java 14a7e9c > ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 7f355e5 > ql/src/test/queries/clientpositive/insert_into1.q f19506a > ql/src/test/results/clientpositive/insert_into1.q.out 9e5f3bb > ql/src/test/results/clientpositive/skewjoin_union_remove_1.q.out 1f21877 > ql/src/test/results/clientpositive/spark/insert_into1.q.out e7f7d8f > ql/src/test/results/clientpositive/stats12.q.out d5ca59b > ql/src/test/results/clientpositive/stats13.q.out e34f0b7 > ql/src/test/results/clientpositive/stats2.q.out 5e305d3 > ql/src/test/results/clientpositive/stats5.q.out 484e834 > ql/src/test/results/clientpositive/stats6.q.out b4435f2 > ql/src/test/results/clientpositive/stats9.q.out 8563f3b > ql/src/test/results/clientpositive/stats_noscan_1.q.out 80c3092 > ql/src/test/results/clientpositive/stats_partscan_1_23.q.out 5488935 > ql/src/test/results/clientpositive/tez/insert_into1.q.out 359470b > ql/src/test/results/clientpositive/tez/stats_noscan_1.q.out aa49526 > ql/src/test/results/clientpositive/union_remove_1.q.out 5fa8caf > ql/src/test/results/clientpositive/union_remove_10.q.out 228ff9e > ql/src/test/results/clientpositive/union_remove_11.q.out bf38f0a > ql/src/test/results/clientpositive/union_remove_12.q.out 3ac1d44 > ql/src/test/results/clientpositive/union_remove_13.q.out b12d7cd > ql/src/test/results/clientpositive/union_remove_14.q.out 94f2ff5 > ql/src/test/results/clientpositive/union_remove_15.q.out 67bdb1e > ql/src/test/results/clientpositive/union_remove_16.q.out 2444337 > ql/src/test/results/clientpositive/union_remove_17.q.out a1b9474 > ql/src/test/results/clientpositive/union_remove_18.q.out 26e52e0 > ql/src/test/results/clientpositive/union_remove_19.q.out d8314f8 > ql/src/test/results/clientpositive/union_remove_2.q.out d3522f2 > ql/src/test/results/clientpositive/union_remove_20.q.out 4503726 > ql/src/test/results/clientpositive/union_remove_21.q.out 98e0166 > ql/src/test/results/clientpositive/union_remove_22.q.out 4f086f3 > ql/src/test/results/clientpositive/union_remove_23.q.out d0e21ba > ql/src/test/results/clientpositive/union_remove_24.q.out f2f6bba > ql/src/test/results/clientpositive/union_remove_25.q.out 3858bdc > ql/src/test/results/clientpositive/union_remove_3.q.out 40e554a > ql/src/test/results/clientpositive/union_remove_4.q.out 10a8e89 > ql/src/test/results/clientpositive/union_remove_5.q.out b59d6da > ql/src/test/results/clientpositive/union_remove_6.q.out 67ba74c > ql/src/test/results/clientpositive/union_remove_6_subq.q.out 0d1a436 > ql/src/test/results/clientpositive/union_remove_7.q.out 76c678f > ql/src/test/results/clientpositive/union_remove_8.q.out fe67448 > ql/src/test/results/clientpositive/union_remove_9.q.out 684091b > > Diff: https://reviews.apache.org/r/34713/diff/ > > > Testing > ------- > > Existing tests + added new tests > > > Thanks, > > Ashutosh Chauhan > >
