[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-11 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r333889584 ## File path: ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java ## @@ -1946,6 +2022,8 @@ public

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-11 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r333889646 ## File path: ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java ## @@ -2039,6 +2117,8 @@ public

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332382671 ## File path: ql/src/test/results/clientpositive/llap/bucket_map_join_tez2.q.out ## @@ -759,50 +759,56 @@ STAGE PLANS:

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332352613 ## File path: ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java ## @@ -276,9 +279,11 @@ public

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332365302 ## File path: ql/src/test/results/clientpositive/llap/retry_failure_stat_changes.q.out ## @@ -139,25 +139,25 @@ Stage-0

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332368069 ## File path: ql/src/test/results/clientpositive/llap/semijoin_reddedup.q.out ## @@ -258,9 +258,12 @@ STAGE PLANS: Tez A masked

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332357556 ## File path: ql/src/java/org/apache/hadoop/hive/ql/stats/StatsUtils.java ## @@ -856,8 +856,15 @@ public static ColStatistics

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332358389 ## File path: ql/src/java/org/apache/hadoop/hive/ql/stats/StatsUtils.java ## @@ -935,8 +942,11 @@ else

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332373527 ## File path:

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332377711 ## File path: standalone-metastore/metastore-common/src/main/thrift/hive_metastore.thrift ## @@ -562,14 +562,27 @@ struct

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332352109 ## File path: common/src/java/org/apache/hadoop/hive/conf/HiveConf.java ## @@ -2537,6 +2537,11 @@ private static void

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332355776 ## File path: ql/src/java/org/apache/hadoop/hive/ql/stats/StatsUtils.java ## @@ -856,8 +856,15 @@ public static ColStatistics

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332353993 ## File path: ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java ## @@ -316,7 +321,7 @@ public

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332383859 ## File path: ql/src/test/results/clientpositive/llap/subquery_select.q.out ## @@ -3918,14 +3918,14 @@ STAGE PLANS:

[GitHub] [hive] kgyrtkirk commented on a change in pull request #787: HIVE-22239

2019-10-08 Thread GitBox
kgyrtkirk commented on a change in pull request #787: HIVE-22239 URL: https://github.com/apache/hive/pull/787#discussion_r332355126 ## File path: ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java ## @@ -967,13 +979,23 @@ private