Skip to content

Conversation

@sun-rui
Copy link
Contributor

@sun-rui sun-rui commented Jul 8, 2015

No description provided.

@viirya
Copy link
Member

viirya commented Jul 8, 2015

@sun-rui I think this is duplicate to #7280.

@SparkQA
Copy link

SparkQA commented Jul 8, 2015

Test build #36795 has finished for PR 7289 at commit 692bd72.

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

@shivaram
Copy link
Contributor

shivaram commented Jul 8, 2015

@sun-rui Lets use the other PR as it also has a test case in it. I think the test case can be improved and you can help review it.

@sun-rui
Copy link
Contributor Author

sun-rui commented Jul 9, 2015

OK

@shivaram
Copy link
Contributor

shivaram commented Jul 9, 2015

@sun-rui Could you close this PR ? Only people who open the PR have permission to close it right now.

@sun-rui sun-rui closed this Aug 29, 2015
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