Skip to content

[CORE] [MINOR] Improve the error message of checkpoint RDD verification#18796

Closed
gatorsmile wants to merge 1 commit intoapache:masterfrom
gatorsmile:improveErrMsgForCheckpoint
Closed

[CORE] [MINOR] Improve the error message of checkpoint RDD verification#18796
gatorsmile wants to merge 1 commit intoapache:masterfrom
gatorsmile:improveErrMsgForCheckpoint

Conversation

@gatorsmile
Copy link
Member

What changes were proposed in this pull request?

The original error message is pretty confusing. It is unable to tell which number is number of partitions and which one is the RDD ID. This PR is to improve the checkpoint checking.

How was this patch tested?

N/A

@gatorsmile
Copy link
Member Author

cc @zsxwing

@dongjoon-hyun
Copy link
Member

+1, LGTM.

@SparkQA
Copy link

SparkQA commented Aug 1, 2017

Test build #80124 has finished for PR 18796 at commit f7524da.

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

@zsxwing
Copy link
Member

zsxwing commented Aug 1, 2017

LGTM

Copy link
Contributor

@jiangxb1987 jiangxb1987 left a comment

Choose a reason for hiding this comment

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

LGTM

@zsxwing
Copy link
Member

zsxwing commented Aug 2, 2017

Thanks! Merging to master

@asfgit asfgit closed this in 4cc704b Aug 2, 2017
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.

5 participants

Comments