Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Jul 21, 2015

This way, the sources package contains only public facing interfaces.

@rxin
Copy link
Contributor Author

rxin commented Jul 21, 2015

cc @liancheng can you take a look at this today? Thanks.

@SparkQA
Copy link

SparkQA commented Jul 21, 2015

Test build #37929 has finished for PR 7565 at commit 9d5196a.

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

@rxin
Copy link
Contributor Author

rxin commented Jul 21, 2015

Why is mima failing for all the private[sql] stuff ... @mengxr any idea?

@liancheng
Copy link
Contributor

LGTM. Jenkins is being shut down. Will retest this after it comes back.

@rxin
Copy link
Contributor Author

rxin commented Jul 21, 2015

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented Jul 21, 2015

Test build #37959 has finished for PR 7565 at commit 7661aff.

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

@asfgit asfgit closed this in 60c0ce1 Jul 21, 2015
@rxin rxin deleted the move-ds branch July 21, 2015 19:03
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.

3 participants