Skip to content

[SPARK-6564][SQL] SQLContext.emptyDataFrame should contain 0 row, not 1 row #5226

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.

Sign up for GitHub

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

Jump to bottom
Closed
rxin wants to merge 2 commits into from
Closed

[SPARK-6564][SQL] SQLContext.emptyDataFrame should contain 0 row, not 1 row #5226

rxin wants to merge 2 commits into from

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Mar 27, 2015

No description provided.

@SparkQA
Copy link

SparkQA commented Mar 27, 2015

Test build #29293 has started for PR 5226 at commit e135bb9.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Mar 27, 2015

Test build #29293 has finished for PR 5226 at commit e135bb9.

  • This patch fails Python style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

AmplabJenkins commented Mar 27, 2015

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29293/
Test FAILed.

@liancheng
Copy link
Contributor

liancheng commented Mar 27, 2015

LGTM

@liancheng
Copy link
Contributor

liancheng commented Mar 27, 2015

retest this please

@AmplabJenkins
Copy link

AmplabJenkins commented Mar 27, 2015

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29301/
Test FAILed.

@SparkQA
Copy link

SparkQA commented Mar 27, 2015

Test build #29315 has started for PR 5226 at commit 1306d88.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Mar 27, 2015

Test build #29315 has finished for PR 5226 at commit 1306d88.

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

@AmplabJenkins
Copy link

AmplabJenkins commented Mar 27, 2015

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29315/
Test PASSed.

@yhuai
Copy link
Contributor

yhuai commented Mar 27, 2015

lgtm

asfgit pushed a commit that referenced this pull request Mar 27, 2015
@rxin
[SPARK-6564][SQL] SQLContext.emptyDataFrame should contain 0 row, not…
7006858 
… 1 row

Author: Reynold Xin <rxin@databricks.com>

Closes #5226 from rxin/empty-df and squashes the following commits:

1306d88 [Reynold Xin] Proper fix.
e135bb9 [Reynold Xin] [SPARK-6564][SQL] SQLContext.emptyDataFrame should contain 0 rows, not 1 row.

(cherry picked from commit 3af7334)
Signed-off-by: Reynold Xin <rxin@databricks.com>
@asfgit asfgit closed this in 3af7334 Mar 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@rxin @SparkQA @AmplabJenkins @liancheng @yhuai