[SPARK-4692] [SQL] Support ! boolean logic operator like NOT

[YanTangZhai]

Support ! boolean logic operator like NOT in sql as follows
select * from for_test where !(col1 > col2)

[SparkQA]

Test build #24041 has started for PR 3555 at commit 92242c7.

• This patch merges cleanly.

[SparkQA]

Test build #24041 has finished for PR 3555 at commit 92242c7.

• 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/24041/
Test PASSed.

[marmbrus]

Please add a test to HiveQuerySuite.

[SparkQA]

Test build #24070 has started for PR 3555 at commit 74175b4.

• This patch merges cleanly.

[SparkQA]

Test build #24070 has finished for PR 3555 at commit 74175b4.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

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

[SparkQA]

Test build #24076 has started for PR 3555 at commit 950b21e.

• This patch merges cleanly.

[SparkQA]

Test build #24076 has finished for PR 3555 at commit 950b21e.

• 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/24076/
Test PASSed.

[marmbrus]

Do other SQL dialects support this? I think we should avoid too much non-standard syntax.

[marmbrus]

How about we switch this to a query test (YanTangZhai#14). otherwise LGTM

[SparkQA]

Test build #24583 has started for PR 3555 at commit 1893956.

• This patch merges cleanly.

[SparkQA]

Test build #24583 has finished for PR 3555 at commit 1893956.

• 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/24583/
Test PASSed.

[YanTangZhai]

@marmbrus Please review again. Thanks.

[AmplabJenkins]

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

[liancheng]

Ah, sorry, forgot that the golden answer file name is generated by the MD5 of the query string. Then let's revert the last space change. I think this minor issue doesn't worth the trouble. Sorry for the inconvenience...

[YanTangZhai]

@liancheng I will revert the last space change. Thanks for your comment.

[SparkQA]

Test build #24702 has started for PR 3555 at commit 1e1ebb4.

• This patch merges cleanly.

[SparkQA]

Test build #24702 has finished for PR 3555 at commit 1e1ebb4.

• 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/24702/
Test PASSed.

[liancheng]

Thanks for the update, this now LGTM.

[marmbrus]

Hey sorry, this has merge conflicts. Can you merge with master?

[SparkQA]

Test build #24935 has started for PR 3555 at commit 192411d.

• This patch does not merge cleanly.

[SparkQA]

Test build #24939 has started for PR 3555 at commit 992046e.

• This patch merges cleanly.

[SparkQA]

Test build #24940 has started for PR 3555 at commit 1a9f605.

• This patch merges cleanly.

[SparkQA]

Test build #24935 has finished for PR 3555 at commit 192411d.

• This patch passes all tests.
• This patch does not merge 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/24935/
Test PASSed.

[YanTangZhai]

@marmbrus I've updated it. Please review again.

[SparkQA]

Test build #24939 has finished for PR 3555 at commit 992046e.

• 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/24939/
Test PASSed.

[SparkQA]

Test build #24940 has finished for PR 3555 at commit 1a9f605.

• 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/24940/
Test PASSed.

[marmbrus]

Thanks! I've merged this to master.