Skip to content

Conversation

@marmbrus
Copy link
Contributor

This behavior has confused some users, so lets clarify it.

Copy link
Member

@gatorsmile gatorsmile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SparkQA
Copy link

SparkQA commented Dec 21, 2017

Test build #85282 has finished for PR 20048 at commit 2f4e7b0.

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

asfgit pushed a commit that referenced this pull request Dec 22, 2017
…oder

This behavior has confused some users, so lets clarify it.

Author: Michael Armbrust <[email protected]>

Closes #20048 from marmbrus/datasetAsDocs.

(cherry picked from commit 8df1da3)
Signed-off-by: gatorsmile <[email protected]>
@gatorsmile
Copy link
Member

Thanks! Merged to master/2.2./2.1

asfgit pushed a commit that referenced this pull request Dec 22, 2017
…oder

This behavior has confused some users, so lets clarify it.

Author: Michael Armbrust <[email protected]>

Closes #20048 from marmbrus/datasetAsDocs.

(cherry picked from commit 8df1da3)
Signed-off-by: gatorsmile <[email protected]>
@asfgit asfgit closed this in 8df1da3 Dec 22, 2017
MatthewRBruce pushed a commit to Shopify/spark that referenced this pull request Jul 31, 2018
…oder

This behavior has confused some users, so lets clarify it.

Author: Michael Armbrust <[email protected]>

Closes apache#20048 from marmbrus/datasetAsDocs.

(cherry picked from commit 8df1da3)
Signed-off-by: gatorsmile <[email protected]>
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