Skip to content

Conversation

@ericl
Copy link
Contributor

@ericl ericl commented Dec 19, 2016

What changes were proposed in this pull request?

This should help prevent accidental incorrect conversions between Path <-> URI.

How was this patch tested?

Existing tests.

@SparkQA
Copy link

SparkQA commented Dec 19, 2016

Test build #70379 has finished for PR 16341 at commit 6e0cc04.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Dec 19, 2016

Test build #70380 has finished for PR 16341 at commit 155f9ab.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Dec 20, 2016

Test build #70394 has finished for PR 16341 at commit bcdac16.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Dec 31, 2016

Test build #70762 has finished for PR 16341 at commit 16b6030.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jan 3, 2017

Test build #70824 has finished for PR 16341 at commit d4f4fe0.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jan 5, 2017

Test build #70934 has finished for PR 16341 at commit 0388ae4.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jan 6, 2017

Test build #70978 has finished for PR 16341 at commit a47e724.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jan 12, 2017

Test build #71236 has finished for PR 16341 at commit 23b30b9.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@ericl ericl closed this Mar 15, 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.

2 participants