[SPARK-4154][SQL] Query does not work if it has "not between " in Spark SQL and HQL

[ravipesala]

if the query contains "not between" does not work like.
SELECT * FROM src where key not between 10 and 20'

[AmplabJenkins]

Can one of the admins verify this patch?

[marmbrus]

ok to test

[SparkQA]

Test build #22540 has started for PR 3017 at commit 805eebc.

• This patch merges cleanly.

[SparkQA]

Test build #22540 has finished for PR 3017 at commit 805eebc.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/22540/
Test PASSed.

[marmbrus]

This is a Nit, but if you have to break the line I think it should be formatted as:

And(
  GreaterThanOrEqual(targetExpression, nodeToExpr(minValue)),
  LessThanOrEqual(targetExpression, nodeToExpr(maxValue)))

This way the tree structure is more clear.

[marmbrus]

Other than a very minor formatting issue LGTM.

[SparkQA]

Test build #22612 has started for PR 3017 at commit 65fc89e.

• This patch merges cleanly.

[ravipesala]

Thank you for your comment.I handled it and also rebased with master.Please review it

[SparkQA]

Test build #22612 has finished for PR 3017 at commit 65fc89e.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/22612/
Test PASSed.

[marmbrus]

Thanks! Merged to master.