Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-10226][SQL] Fix exclamation mark issue in SparkSQL #8420

Closed
wants to merge 1 commit into from

Conversation

small-wang
Copy link
Contributor

When I tested the latest version of spark with exclamation mark, I got some errors. Then I reseted the spark version and found that commit id "a2409d1c8e8ddec04b529ac6f6a12b5993f0eeda" brought the bug. With jline version changing from 0.9.94 to 2.12 after this commit, exclamation mark would be treated as a special character in ConsoleReader.

@rxin
Copy link
Contributor

rxin commented Aug 25, 2015

cc @dragos can you take a look at this?

@rxin
Copy link
Contributor

rxin commented Aug 25, 2015

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Aug 25, 2015

Test build #41558 has finished for PR 8420 at commit 2e178d9.

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

@watermen
Copy link
Contributor

LGTM @dragos

@dragos
Copy link
Contributor

dragos commented Aug 28, 2015

Sorry, I missed the Github notification. LGTM. The Scala REPL does the same thing. Probably the ancient version of jline that was used before didn't perform event expansion.

@marmbrus
Copy link
Contributor

Thanks, I'm going to merge this into master and 1.5

asfgit pushed a commit that referenced this pull request Aug 29, 2015
When I tested the latest version of spark with exclamation mark, I got some errors. Then I reseted the spark version and found that commit id "a2409d1c8e8ddec04b529ac6f6a12b5993f0eeda" brought the bug. With jline version changing from 0.9.94 to 2.12 after this commit, exclamation mark would be treated as a special character in ConsoleReader.

Author: wangwei <wangwei82@huawei.com>

Closes #8420 from small-wang/jline-SPARK-10226.

(cherry picked from commit 277148b)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@asfgit asfgit closed this in 277148b Aug 29, 2015
@small-wang small-wang deleted the jline-SPARK-10226 branch December 10, 2015 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants