Skip to content

Conversation

@marmbrus
Copy link
Contributor

@marmbrus marmbrus commented Aug 5, 2015

Make sure that $"column" is consistent with other methods with respect to backticks. Adds a bunch of tests for various ways of constructing columns.

@JoshRosen
Copy link
Contributor

I wonder if I could extend my fuzz tester to cover these cases...

@SparkQA
Copy link

SparkQA commented Aug 5, 2015

Test build #39899 has finished for PR 7969 at commit 53ef3d7.

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

@marmbrus
Copy link
Contributor Author

marmbrus commented Aug 5, 2015

test this please

@SparkQA
Copy link

SparkQA commented Aug 6, 2015

Test build #39942 has finished for PR 7969 at commit 53ef3d7.

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

@rxin
Copy link
Contributor

rxin commented Aug 6, 2015

cc @cloud-fan for review

@cloud-fan
Copy link
Contributor

LGTM

@rxin
Copy link
Contributor

rxin commented Aug 7, 2015

Merging this in.

asfgit pushed a commit that referenced this pull request Aug 7, 2015
Make sure that `$"column"` is consistent with other methods with respect to backticks.  Adds a bunch of tests for various ways of constructing columns.

Author: Michael Armbrust <michael@databricks.com>

Closes #7969 from marmbrus/namesWithDots and squashes the following commits:

53ef3d7 [Michael Armbrust] [SPARK-9650][SQL] Fix quoting behavior on interpolated column names
2bf7a92 [Michael Armbrust] WIP

(cherry picked from commit 0867b23)
Signed-off-by: Reynold Xin <rxin@databricks.com>
@asfgit asfgit closed this in 0867b23 Aug 7, 2015
@marmbrus marmbrus deleted the namesWithDots branch March 8, 2016 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants