Skip to content

SKIPME expose the parquet util to be public to be used by csd persist… #71

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.

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

Closed
wants to merge 1 commit into from

Conversation

wli600
Copy link

@wli600 wli600 commented Jul 28, 2015

replacing the one on master, #70

@davidnavas @markhamstra

was able to run the test unit with the snapshot build with corresponding changes

@davidnavas
Copy link

Seems reasonable -- why all the failures though?

@wli600
Copy link
Author

wli600 commented Jul 29, 2015

jenkins, test this please

@markhamstra
Copy link

I wouldn't expect Jenkins to be able to run the Spark tests consistently. There are some known flakey tests and some differences between how our Jenkins runs the tests vs. how the canonical AMPLab Jenkins does. I'll run them locally and see if there is anything to be concerned over.

@wli600
Copy link
Author

wli600 commented Jul 29, 2015

sweet, thanks, Mark, yeah, these changes should not be harmful from the
look of it,

meanwhile, I think I just kicked off a jenkins job, finger crossed,

On Wed, Jul 29, 2015 at 9:41 AM, Mark Hamstra notifications@github.com
wrote:

I wouldn't expect Jenkins to be able to run the Spark tests consistently.
There are some known flakey tests and some differences between how our
Jenkins runs the tests vs. how the canonical AMPLab Jenkins does. I'll run
them locally and see if there is anything to be concerned over.


Reply to this email directly or view it on GitHub
#71 (comment).

  • Wei

@wli600 wli600 closed this Jul 29, 2015
@wli600 wli600 deleted the parquet-1.4 branch July 29, 2015 18:06
@wli600
Copy link
Author

wli600 commented Jul 29, 2015

replaced with proper branch, #73

markhamstra pushed a commit to markhamstra/spark that referenced this pull request Nov 7, 2017
* Allow adding arbitrary files

* Address comments and add documentation
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