[SPARK-6185][SQL] Deltele repeated TOKEN. "TOK_CREATEFUNCTION" and "TOK_SHOWINDEXES"

[pzzs]

Line 65 "TOK_SHOWINDEXES"
Line 66 "TOK_SHOWINDEXES"

Line 84 "TOK_CREATEFUNCTION"
Line 106 "TOK_CREATEFUNCTION"

[AmplabJenkins]

Can one of the admins verify this patch?

[chenghao-intel]

ok to test

[chenghao-intel]

Good catch, LGTM.

[adrian-wang]

Maybe we can sort all these tokens in alphabetic order to avoid duplicate.

[pzzs]

Anyone could test it? @chenghao-intel @marmbrus

[yhuai]

ok to test

[SparkQA]

Test build #28381 has started for PR 4907 at commit c0bd3f9.

• This patch merges cleanly.

[SparkQA]

Test build #28381 has finished for PR 4907 at commit c0bd3f9.

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

[pzzs]

@yhuai Could it be merged?

[adrian-wang]

Again, I think we'd better sort these tokens in alphabetic order to avoid further duplicate

[marmbrus]

+1 to sorting.
On Mar 10, 2015 7:26 PM, "Daoyuan Wang" notifications@github.com wrote:

Again, I think we'd better sort these tokens in alphabetic order to avoid
further duplicate

—
Reply to this email directly or view it on GitHub
#4907 (comment).

[pzzs]

It is a good idea that to sorting , i will do it @adrian-wang @marmbrus @chenghao-intel

[pzzs]

I hava opened a new pr for sorting these tokens in alphabetic.
#4973 @yhuai @marmbrus

[srowen]

Mind closing this PR?

[pzzs]

yes i will close it @srowen