[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs

2018-01-31 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16348135#comment-16348135 ] Hive QA commented on HIVE-18552: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs

2018-01-31 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16348087#comment-16348087 ] Hive QA commented on HIVE-18552: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs

2018-01-31 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16347653#comment-16347653 ] Vihang Karajgaonkar commented on HIVE-18552: patch looks good to me. +1 Since this removes an

[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs

2018-01-31 Thread Sahil Takiar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16347327#comment-16347327 ] Sahil Takiar commented on HIVE-18552: - [~aihuaxu], [~vihangk1] could you take a look? > Split

[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs

2018-01-30 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16346351#comment-16346351 ] Hive QA commented on HIVE-18552: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs

2018-01-30 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16346317#comment-16346317 ] Hive QA commented on HIVE-18552: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs

2018-01-29 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16344510#comment-16344510 ] Hive QA commented on HIVE-18552: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs

2018-01-29 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16344475#comment-16344475 ] Hive QA commented on HIVE-18552: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs, enable partition filter check by default

2018-01-29 Thread Sahil Takiar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16344366#comment-16344366 ] Sahil Takiar commented on HIVE-18552: - [~sershe] sounds reasonable. Didn't realize there was an

[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs, enable partition filter check by default

2018-01-25 Thread Sergey Shelukhin (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16340441#comment-16340441 ] Sergey Shelukhin commented on HIVE-18552: - It makes sense to separate them, but requiring a

[jira] [Commented] (HIVE-18552) Split hive.strict.checks.large.query into two configs, enable partition filter check by default

2018-01-25 Thread Sahil Takiar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16340428#comment-16340428 ] Sahil Takiar commented on HIVE-18552: - [~sershe], [~ashutoshc] any thoughts on this? > Split