Revert [SPARK-27142][SPARK-31440] SQL rest API in branch 3.0

[gengliangwang]

What changes were proposed in this pull request?

Revert #28208 and #24076 in branch 3.0

Why are the changes needed?

Unfortunately, the PR #28208 is merged after Spark 3.0 RC 2 cut. Although the improvement is great, we can't break the policy to add new improvement commits into branch 3.0 now.

Also, if we are going to adopt the improvement in a future release, we should not release 3.0 with #24076, since the API result will be changed.

After discuss with @cloud-fan and @gatorsmile offline, we think the best choice is to revert both commits and follow community release policy.

Does this PR introduce any user-facing change?

Yes, let's hold the SQL rest API until next release.

How was this patch tested?

Jenkins unit tests.

[gengliangwang]

cc @erenavsarogullari @ajithme as well

[ajithme]

agree. Its better to revert and we can reraise the PR on later release. will 3.x be a candidate for these PRs.?

[gengliangwang]

@ajithme thanks for understanding! The commits are still in the master branch and they will be 3.1 as well.

[cloud-fan]

It's unfortunate that we find problems in SQL REST API at this stage which is too late to change. LGTM to revert it from 3.0 and release it in 3.1.

[erenavsarogullari]

Hi @gengliangwang,
Sure, make sense so end-user can start to use single version(3.1) directly without backward compatibility requirement. Thanks for the effort.

[gengliangwang]

merging to 3.0

[HyukjinKwon]

@gengliangwang test is still running

[HyukjinKwon]

+1 from my side too.

[gengliangwang]

@HyukjinKwon oh thanks for reminder

[gatorsmile]

LGTM

[SparkQA]

Test build #122871 has finished for PR 28588 at commit 9e39968.

• This patch fails due to an unknown error code, -9.
• This patch merges cleanly.
• This patch adds no public classes.

[gengliangwang]

retest this please.

[SparkQA]

Test build #122876 has finished for PR 28588 at commit 9e39968.

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

[cloud-fan]

thanks, merging to 3.0!