Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19631
Merging to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/19631
+1
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/19631
> If I understand what you're saying correctly, that should be considered a
security issue in that server application regardless of this change. The server
should not be exposing its environment
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19631
LGTM.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84363/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #84363 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84363/testReport)**
for PR 19631 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #84363 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84363/testReport)**
for PR 19631 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19631
> For instance if someone is starting a server that is getting hit by other
users that server could be started with the same env and then inadvertently
expose the secret to other users.
If
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/19631
fyi - in progress of reviewing should be done in an hour or so.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83951/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83951 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83951/testReport)**
for PR 19631 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83951 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83951/testReport)**
for PR 19631 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83946/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83946 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83946/testReport)**
for PR 19631 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83946 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83946/testReport)**
for PR 19631 at commit
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/19631
Sorry on vacation til Monday. I would like to look at this though since it
says changing secret passing to env variable.
---
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19631
Did another round of review, LGTM overall. @tgravescs do you any comment?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83876/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83876 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83876/testReport)**
for PR 19631 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83875/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83875 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83875/testReport)**
for PR 19631 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83876 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83876/testReport)**
for PR 19631 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83875 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83875/testReport)**
for PR 19631 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83858/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83858 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83858/testReport)**
for PR 19631 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83858 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83858/testReport)**
for PR 19631 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19631
@tgravescs @jerryshao
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83555/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83555 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83555/testReport)**
for PR 19631 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83555 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83555/testReport)**
for PR 19631 at commit
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/19631
That's an expected message from the build and has no impact on it. I'll
change the build config to suppress the message when I'm back from holiday
late next week.
On Nov 1, 2017
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83352/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83352 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83352/testReport)**
for PR 19631 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83352 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83352/testReport)**
for PR 19631 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19631
Sigh. retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83349/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19631
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83305 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83305/testReport)**
for PR 19631 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19631
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83302/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83299/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83299 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83299/testReport)**
for PR 19631 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19631
**[Test build #83299 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83299/testReport)**
for PR 19631 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19631
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19631
@shaneknapp any idea what's going on?
```
chmod: cannot access `target/*': No such file or directory
```
---
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83298/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19631
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19631
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83297/
Test FAILed.
---
61 matches
Mail list logo