Skip to content

Conversation

@HyukjinKwon
Copy link
Member

@HyukjinKwon HyukjinKwon commented Jun 27, 2017

What changes were proposed in this pull request?

I tried to test with lower versions of conda (3.6.0 on Ubuntu 14.04). I could reproduce a similar error with this.

This PR proposes to disable SSL verification with conda to unblock other PRs.

How was this patch tested?

Manually tested via shell.

@HyukjinKwon HyukjinKwon changed the title [WIP][HOTFIX] Print out a wraning if pyarrow is not installed and ignore the error [WIP][HOTFIX] Disable SSL verification to install pyarrow with conda Jun 27, 2017
@HyukjinKwon
Copy link
Member Author

retest this please

1 similar comment
@HyukjinKwon
Copy link
Member Author

retest this please

@HyukjinKwon HyukjinKwon changed the title [WIP][HOTFIX] Disable SSL verification to install pyarrow with conda [HOTFIX] Disable SSL verification to install pyarrow with conda Jun 27, 2017
@SparkQA
Copy link

SparkQA commented Jun 27, 2017

Test build #78715 has finished for PR 18439 at commit 270b6a3.

  • This patch fails due to an unknown error code, -10.
  • This patch merges cleanly.
  • This patch adds no public classes.

@HyukjinKwon
Copy link
Member Author

cc @holdenk and @shaneknapp, what do you think about this change?

I at least see Running PySpark packaging tests is passed for three builds above.

@holdenk
Copy link
Contributor

holdenk commented Jun 27, 2017

If @shaneknapp has the bandwidth to update conda (or is ok with me updating conda) I would feel more comfortable doing that than turning of SSL verification.

@holdenk
Copy link
Contributor

holdenk commented Jun 27, 2017

I'm also in favour of backing out the original PR until we can resolve the issue, but I'll defer to @JoshRosen on this.

@SparkQA
Copy link

SparkQA commented Jun 27, 2017

Test build #78714 has finished for PR 18439 at commit 270b6a3.

  • This patch fails due to an unknown error code, -10.
  • This patch merges cleanly.
  • This patch adds no public classes.

@shaneknapp
Copy link
Contributor

i've installed pyarrow 0.4.0 on the workers in the py3k conda environment, and would not like to disable SSL.

i do not approve of this PR, sorry.

@SparkQA
Copy link

SparkQA commented Jun 27, 2017

Test build #78712 has finished for PR 18439 at commit 270b6a3.

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

@HyukjinKwon
Copy link
Member Author

So, is this resolved now without this chsnge?

@HyukjinKwon
Copy link
Member Author

Ok nvm. I assume we don't want to fix it in this way and going to unblock other PRs soon. I am closing this.

@shaneknapp
Copy link
Contributor

shaneknapp commented Jun 27, 2017 via email

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