Skip to content

[SPARK-11746][CORE]Use cache-aware method dependencies #9691

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
suyanNone wants to merge 1 commit into from
Closed

[SPARK-11746][CORE]Use cache-aware method dependencies #9691

suyanNone wants to merge 1 commit into from
+1 −1

Conversation

@suyanNone
Copy link
Contributor

@suyanNone suyanNone commented Nov 13, 2015

a small change

@SparkQA
Copy link

SparkQA commented Nov 13, 2015

Test build #45860 has finished for PR 9691 at commit 4ac866f.

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

@andrewor14
Copy link
Contributor

andrewor14 commented Nov 13, 2015

please file JIRA and explain why you're doing this

@srowen
Copy link
Member

srowen commented Nov 13, 2015

I think that's probably right, as it is a "checkpoint aware and caching" getter for getDependencies, but man that is confusing.

@rxin
Copy link
Contributor

rxin commented Nov 13, 2015

@suyanNone can you create a JIRA ticket for this?

@rxin
Copy link
Contributor

rxin commented Nov 13, 2015

The change itself LGTM.

@suyanNone suyanNone changed the title Unify dependencies entry [SPARK-11746][CORE]Use cache-aware method dependencies Nov 16, 2015
@andrewor14
Copy link
Contributor

andrewor14 commented Nov 19, 2015

Merged into master 1.6.

asfgit pushed a commit that referenced this pull request Nov 19, 2015
@suyanNone
[SPARK-11746][CORE] Use cache-aware method dependencies
baae1cc 
a small change

Author: hushan <[email protected]>

Closes #9691 from suyanNone/unify-getDependency.

(cherry picked from commit 01403aa)
Signed-off-by: Andrew Or <[email protected]>
@asfgit asfgit closed this in 01403aa Nov 19, 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

@suyanNone @SparkQA @andrewor14 @srowen @rxin