Skip to content

Conversation

@witgo
Copy link
Contributor

@witgo witgo commented Aug 21, 2014

No description provided.

@SparkQA
Copy link

SparkQA commented Aug 21, 2014

QA tests have started for PR 2075 at commit 5f104d0.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 21, 2014

QA tests have finished for PR 2075 at commit 5f104d0.

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

@srowen
Copy link
Member

srowen commented Aug 21, 2014

I think this is the same issue already addressed by https://github.com/apache/spark/pull/1726/files

@SparkQA
Copy link

SparkQA commented Aug 21, 2014

QA tests have started for PR 2075 at commit b2f3747.

  • This patch does not merge cleanly!

@SparkQA
Copy link

SparkQA commented Aug 21, 2014

QA tests have finished for PR 2075 at commit b2f3747.

  • This patch passes unit tests.
  • This patch does not merge cleanly!

@tdas
Copy link
Contributor

tdas commented Aug 23, 2014

Hey @witgo
Thanks for your PR, that should have fixed the maven build issue. However, I looked through the code and found a better fix. I submitted it as a PR here
#2101

So, can you close this PR? Thank for the effort!

@witgo witgo closed this Aug 23, 2014
@witgo witgo deleted the SPARK-3169 branch August 23, 2014 02:24
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