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-12340][SQL]fix Int overflow in the SparkPlan.executeTake, RDD.take and AsyncRDDActions.takeAsync #10487

Closed
wants to merge 3 commits into from

Conversation

QiangCai
Copy link
Contributor

@srowen I new this pull request to to resolve the problem.

another pull request
#10310

jira
https://issues.apache.org/jira/browse/SPARK-12340

@SparkQA
Copy link

SparkQA commented Dec 27, 2015

Test build #2256 has finished for PR 10487 at commit 18f1b81.

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

@srowen
Copy link
Member

srowen commented Dec 28, 2015

LGTM

@sarutak
Copy link
Member

sarutak commented Dec 28, 2015

@QiangCai LGTM but could you add test cases?

@QiangCai
Copy link
Contributor Author

@sarutak I will try to add test cases.

@QiangCai
Copy link
Contributor Author

QiangCai commented Jan 3, 2016

When I am rebasing from master, I get many conflicts. I don't known how to resolve it. I have just pushed the commit "merge".

@srowen
Copy link
Member

srowen commented Jan 3, 2016

@QiangCai the merge commit is wrong as you can see. You can revert it, commit, rebase from master, fix conflicts, and force-push a merge to fix. Or, if you're not sure how to do it, close this PR and open a new one.

@QiangCai
Copy link
Contributor Author

QiangCai commented Jan 3, 2016

I have created another pull request #10562.

@QiangCai QiangCai closed this Jan 3, 2016
asfgit pushed a commit that referenced this pull request Jan 6, 2016
….take and AsyncRDDActions.takeAsync

I have closed pull request #10487. And I create this pull request to resolve the problem.

spark jira
https://issues.apache.org/jira/browse/SPARK-12340

Author: QiangCai <[email protected]>

Closes #10562 from QiangCai/bugfix.
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.

4 participants