Skip to content

Conversation

@radek1st
Copy link
Contributor

[SQL][DOC] I found it a bit confusing when I came across it for the first time in the docs

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@radek1st radek1st changed the title improved a comment [SQL][DOC]improved a comment May 21, 2015
@radek1st radek1st changed the title [SQL][DOC]improved a comment [SQL][DOC] improved a comment May 21, 2015
@srowen
Copy link
Member

srowen commented May 21, 2015

It doesn't seem that different, so hard to object to, but what does this really clarify?

@radek1st
Copy link
Contributor Author

I agree, it's no biggy, but the method signature says select(Column) and not selectExpr(...), so the original comment doesn't seem valid on the first sight. I'm still new to Spark so it's for you to decide, but I found it a bit confusing

@radek1st
Copy link
Contributor Author

btw, I like your anomaly detection talk :)

asfgit pushed a commit that referenced this pull request Jun 16, 2015
[SQL][DOC] I found it a bit confusing when I came across it for the first time in the docs

Author: Radek Ostrowski <[email protected]>
Author: radek <[email protected]>

Closes #6332 from radek1st/master and squashes the following commits:

dae3347 [Radek Ostrowski] fixed typo
c76bb3a [radek] improved a comment

(cherry picked from commit 4bd10fd)
Signed-off-by: Sean Owen <[email protected]>
@asfgit asfgit closed this in 4bd10fd Jun 16, 2015
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
[SQL][DOC] I found it a bit confusing when I came across it for the first time in the docs

Author: Radek Ostrowski <[email protected]>
Author: radek <[email protected]>

Closes apache#6332 from radek1st/master and squashes the following commits:

dae3347 [Radek Ostrowski] fixed typo
c76bb3a [radek] improved a comment
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.

3 participants