Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21956
@HyukjinKwon sure, I will open a PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21956
@kiszk, I would appreciate if you feel free to open a PR fixing them, or
suggest them in someone's PR fixing them.
---
-
To
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21956
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user deshanxiao commented on the issue:
https://github.com/apache/spark/pull/21956
It's ok. Thank you for your suggestions!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21956
Would it be possible to address the following parts for `can ...`, `could
...`, `may ...`, and `will ...`, too?
`JavaSparkContextSuite.scala`
```
/**
* Java apps can uses both
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21956
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/21956
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94212/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21956
**[Test build #94212 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94212/testReport)**
for PR 21956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21956
**[Test build #94212 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94212/testReport)**
for PR 21956 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21956
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/21956
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/21956
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94198/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21956
**[Test build #94198 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94198/testReport)**
for PR 21956 at commit
Github user deshanxiao commented on the issue:
https://github.com/apache/spark/pull/21956
Thanks a lot, I am sorry that I'm not able to find more clear errors. As a
beginner, I am very glad to write more docs to help others understanding the
spark core easier.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21956
**[Test build #94198 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94198/testReport)**
for PR 21956 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21956
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21956
It's okay @deshanxiao but mind taking another look and see if there are
other typos in this module? I am pretty sure there are more and it should be
good to fix them in a batch while we are
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21956
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21956
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21956
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user holdensmagicalunicorn commented on the issue:
https://github.com/apache/spark/pull/21956
@deshanxiao, thanks! I am a bot who has found some folks who might be able
to help with the review:@JoshRosen, @rxin and @cloud-fan
---
21 matches
Mail list logo