[SPARK-14133][SQL] Throws exception for unsupported create/drop/alter index , and lock/unlock operations.

[sureshthalamati]

What changes were proposed in this pull request?

This PR throws Unsupported Operation exception for create index, drop index, alter index , lock table , lock database, unlock table, and unlock database operations that are not supported in Spark SQL. Currently these operations are executed executed by Hive.

Error:
spark-sql> drop index my_index on my_table;
Error in query:
Unsupported operation: drop index(line 1, pos 0)

How was this patch tested?

Added test cases to HiveQuerySuite

@yhuai @hvanhovell @andrewor14

[sureshthalamati]

This Jira also mentions alter table touch as an unsupported operation. But there is an explicit parser rule for this command , I was not sure if there is work in progress to support it or not ? If there are no plans to support I will update this PR accordingly.

[hvanhovell]

Is this a non-reserved keyword according to Hive? If it is please add it to the nonReserved rule.

[hvanhovell]

It is non-Reserved.

[sureshthalamati]

Thank you for reviewing the PR, Herman. Added it to the non-Reserved list.

[hvanhovell]

@sureshthalamati thanks for working on this. The ALTER TABLE table TOUCH [PARTITION spec] command has already been implemented.

[SparkQA]

Test build #2709 has finished for PR 12069 at commit 6f46e32.

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

[sureshthalamati]

Thanks for the input, Herman. Addressed the review comments , and fixed the test failure in my latest commit.

[andrewor14]

@sureshthalamati can you rebase now?

[sureshthalamati]

Thanks for reviewing the PR , Andrew. I rebased it.

[andrewor14]

retest this please

[SparkQA]

Test build #54655 has finished for PR 12069 at commit 31ae9d5.

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

[sureshthalamati]

The failed test in previous run : org.apache.spark.sql.sources.JsonHadoopFsRelationSuite passes for me on my local machine.

[hvanhovell]

Yeah that is a bit funny. I triggered another build.

[SparkQA]

Test build #2719 has finished for PR 12069 at commit 31ae9d5.

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

[hvanhovell]

LGTM

[hvanhovell]

Merging to master. Thanks!

[sureshthalamati]

Thanks, Herman.