[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-06 Thread Hive QA (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16834365#comment-16834365
 ] 

Hive QA commented on HIVE-21685:




Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12967952/HIVE-21685.patch

{color:green}SUCCESS:{color} +1 due to 1 test(s) being added or modified.

{color:green}SUCCESS:{color} +1 due to 15981 tests passed

Test results: 
https://builds.apache.org/job/PreCommit-HIVE-Build/17143/testReport
Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/17143/console
Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-17143/

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.TestCheckPhase
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.YetusPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12967952 - PreCommit-HIVE-Build

> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch, HIVE-21685.patch, HIVE-21685.patch, 
> HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-06 Thread Hive QA (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16834346#comment-16834346
 ] 

Hive QA commented on HIVE-21685:


| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  1m 
52s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  7m 
 0s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m  
4s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
39s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  4m  
1s{color} | {color:blue} ql in master has 2253 extant Findbugs warnings. 
{color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
56s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
27s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  1m 
24s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m  
4s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  1m  
4s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  0m 
43s{color} | {color:red} ql: The patch generated 3 new + 87 unchanged - 0 fixed 
= 90 total (was 87) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  4m 
10s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
59s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
13s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 25m  4s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Optional Tests |  asflicense  javac  javadoc  findbugs  checkstyle  compile  |
| uname | Linux hiveptest-server-upstream 3.16.0-4-amd64 #1 SMP Debian 
3.16.43-2+deb8u5 (2017-09-19) x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/data/hiveptest/working/yetus_PreCommit-HIVE-Build-17143/dev-support/hive-personality.sh
 |
| git revision | master / c5e6608 |
| Default Java | 1.8.0_111 |
| findbugs | v3.0.0 |
| checkstyle | 
http://104.198.109.242/logs//PreCommit-HIVE-Build-17143/yetus/diff-checkstyle-ql.txt
 |
| modules | C: ql itests U: . |
| Console output | 
http://104.198.109.242/logs//PreCommit-HIVE-Build-17143/yetus.txt |
| Powered by | Apache Yetushttp://yetus.apache.org |


This message was automatically generated.



> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch, HIVE-21685.patch, HIVE-21685.patch, 
> HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-04 Thread Zoltan Haindrich (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16833122#comment-16833122
 ] 

Zoltan Haindrich commented on HIVE-21685:
-

yes; I think it's redundant - but it can be removed later as well
+1

> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch, HIVE-21685.patch, HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-03 Thread Hive QA (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16832663#comment-16832663
 ] 

Hive QA commented on HIVE-21685:




Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12967792/HIVE-21685.patch

{color:green}SUCCESS:{color} +1 due to 1 test(s) being added or modified.

{color:red}ERROR:{color} -1 due to 1 failed/errored test(s), 15972 tests 
executed
*Failed tests:*
{noformat}
org.apache.hadoop.hive.ql.parse.TestReplicationScenariosAcidTablesBootstrap.testBootstrapAcidTablesDuringIncrementalWithConcurrentWrites
 (batchId=245)
{noformat}

Test results: 
https://builds.apache.org/job/PreCommit-HIVE-Build/17114/testReport
Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/17114/console
Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-17114/

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.TestCheckPhase
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.YetusPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 1 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12967792 - PreCommit-HIVE-Build

> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch, HIVE-21685.patch, HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-03 Thread Hive QA (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16832626#comment-16832626
 ] 

Hive QA commented on HIVE-21685:


| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  1m 
57s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  7m 
25s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 
11s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
40s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  4m 
12s{color} | {color:blue} ql in master has 2253 extant Findbugs warnings. 
{color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m  
2s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
30s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  1m 
30s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m  
5s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  1m  
5s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  0m 
43s{color} | {color:red} ql: The patch generated 3 new + 87 unchanged - 0 fixed 
= 90 total (was 87) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  4m 
14s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m  
5s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
14s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 26m 17s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Optional Tests |  asflicense  javac  javadoc  findbugs  checkstyle  compile  |
| uname | Linux hiveptest-server-upstream 3.16.0-4-amd64 #1 SMP Debian 
3.16.43-2+deb8u5 (2017-09-19) x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/data/hiveptest/working/yetus_PreCommit-HIVE-Build-17114/dev-support/hive-personality.sh
 |
| git revision | master / 7ae14d1 |
| Default Java | 1.8.0_111 |
| findbugs | v3.0.0 |
| checkstyle | 
http://104.198.109.242/logs//PreCommit-HIVE-Build-17114/yetus/diff-checkstyle-ql.txt
 |
| modules | C: ql itests U: . |
| Console output | 
http://104.198.109.242/logs//PreCommit-HIVE-Build-17114/yetus.txt |
| Powered by | Apache Yetushttp://yetus.apache.org |


This message was automatically generated.



> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch, HIVE-21685.patch, HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-03 Thread Jesus Camacho Rodriguez (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16832603#comment-16832603
 ] 

Jesus Camacho Rodriguez commented on HIVE-21685:


[~kgyrtkirk], we can do that. I assume what you mean is that now that we 
translate IN clauses into AND/OR, we would end up simplifying the predicate in 
any case. The only advantage that I observe, if we keep this logic, is that we 
may be able to make expressions more compact/simplify them before going into 
the expressions reducer, which is a more expensive step (probably noticeable 
only for queries with complex expressions).

Nevertheless, do you think it is valuable to push this patch since the fix is 
obvious, then tackle removal in follow-up? This seemed to be an issue for a 
while and it will be easier to backport since it does not depend on any other 
features/fixes.

> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch, HIVE-21685.patch, HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-03 Thread Zoltan Haindrich (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16832300#comment-16832300
 ] 

Zoltan Haindrich commented on HIVE-21685:
-

I think we should could also consider removing the RexMergeInClause ; I think 
it's not that usefull anymore since we are opening/closing INs.


> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch, HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-02 Thread Hive QA (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16832254#comment-16832254
 ] 

Hive QA commented on HIVE-21685:




Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12967733/HIVE-21685.patch

{color:green}SUCCESS:{color} +1 due to 1 test(s) being added or modified.

{color:red}ERROR:{color} -1 due to 1 failed/errored test(s), 15909 tests 
executed
*Failed tests:*
{noformat}
TestContribCliDriver - did not produce a TEST-*.xml file (likely timed out) 
(batchId=278)
{noformat}

Test results: 
https://builds.apache.org/job/PreCommit-HIVE-Build/17107/testReport
Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/17107/console
Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-17107/

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.TestCheckPhase
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.YetusPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 1 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12967733 - PreCommit-HIVE-Build

> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch, HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-02 Thread Hive QA (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16832237#comment-16832237
 ] 

Hive QA commented on HIVE-21685:


| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  1m 
49s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  7m 
28s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m  
6s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
40s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  4m 
13s{color} | {color:blue} ql in master has 2253 extant Findbugs warnings. 
{color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m  
1s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
28s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  1m 
26s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m  
8s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  1m  
8s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  0m 
42s{color} | {color:red} ql: The patch generated 3 new + 87 unchanged - 0 fixed 
= 90 total (was 87) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  4m 
25s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m  
0s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
16s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 26m 16s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Optional Tests |  asflicense  javac  javadoc  findbugs  checkstyle  compile  |
| uname | Linux hiveptest-server-upstream 3.16.0-4-amd64 #1 SMP Debian 
3.16.43-2+deb8u5 (2017-09-19) x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/data/hiveptest/working/yetus_PreCommit-HIVE-Build-17107/dev-support/hive-personality.sh
 |
| git revision | master / 9bf1869 |
| Default Java | 1.8.0_111 |
| findbugs | v3.0.0 |
| checkstyle | 
http://104.198.109.242/logs//PreCommit-HIVE-Build-17107/yetus/diff-checkstyle-ql.txt
 |
| modules | C: ql itests U: . |
| Console output | 
http://104.198.109.242/logs//PreCommit-HIVE-Build-17107/yetus.txt |
| Powered by | Apache Yetushttp://yetus.apache.org |


This message was automatically generated.



> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch, HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-02 Thread Hive QA (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16832160#comment-16832160
 ] 

Hive QA commented on HIVE-21685:




Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12967728/HIVE-21685.patch

{color:green}SUCCESS:{color} +1 due to 1 test(s) being added or modified.

{color:red}ERROR:{color} -1 due to 1 failed/errored test(s), 15880 tests 
executed
*Failed tests:*
{noformat}
org.apache.hadoop.hive.ql.parse.TestReplWithJsonMessageFormat.org.apache.hadoop.hive.ql.parse.TestReplWithJsonMessageFormat
 (batchId=246)
{noformat}

Test results: 
https://builds.apache.org/job/PreCommit-HIVE-Build/17104/testReport
Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/17104/console
Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-17104/

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.TestCheckPhase
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.YetusPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 1 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12967728 - PreCommit-HIVE-Build

> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HIVE-21685) Wrong simplification in query with multiple IN clauses

2019-05-02 Thread Hive QA (JIRA)


[ 
https://issues.apache.org/jira/browse/HIVE-21685?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16832153#comment-16832153
 ] 

Hive QA commented on HIVE-21685:


| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  1m 
52s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  7m 
30s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 
11s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
41s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  4m  
5s{color} | {color:blue} ql in master has 2253 extant Findbugs warnings. 
{color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m  
4s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
28s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  1m 
29s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m  
9s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  1m  
9s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  0m 
39s{color} | {color:red} ql: The patch generated 3 new + 87 unchanged - 0 fixed 
= 90 total (was 87) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  4m 
18s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m  
0s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
14s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 26m 13s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Optional Tests |  asflicense  javac  javadoc  findbugs  checkstyle  compile  |
| uname | Linux hiveptest-server-upstream 3.16.0-4-amd64 #1 SMP Debian 
3.16.43-2+deb8u5 (2017-09-19) x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/data/hiveptest/working/yetus_PreCommit-HIVE-Build-17104/dev-support/hive-personality.sh
 |
| git revision | master / 9bf1869 |
| Default Java | 1.8.0_111 |
| findbugs | v3.0.0 |
| checkstyle | 
http://104.198.109.242/logs//PreCommit-HIVE-Build-17104/yetus/diff-checkstyle-ql.txt
 |
| modules | C: ql itests U: . |
| Console output | 
http://104.198.109.242/logs//PreCommit-HIVE-Build-17104/yetus.txt |
| Powered by | Apache Yetushttp://yetus.apache.org |


This message was automatically generated.



> Wrong simplification in query with multiple IN clauses
> --
>
> Key: HIVE-21685
> URL: https://issues.apache.org/jira/browse/HIVE-21685
> Project: Hive
>  Issue Type: Bug
>  Components: CBO
>Reporter: Oliver Draese
>Assignee: Jesus Camacho Rodriguez
>Priority: Major
> Attachments: HIVE-21685.patch
>
>
> Simple test to reproduce:
> {code}
> select * from table1 where name IN(‘g’,‘r’) AND name IN(‘a’,‘b’);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)