Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22772
Thank you @gatorsmile @HyukjinKwon @xuanyuanking
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22772
LGTM
Let us merge this PR first. If we find more, feel free to open a new
follow-up PR
---
-
To unsubscribe,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22772
It's okay. the doc fix was huge and there should likely be some mistakes. I
will read it closely too this weekends.
---
-
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22772
@HyukjinKwon Sure..
---
-
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/22772
Minor fixes are minor .. no need to rush to get this into 2.4. Let's take a
look few more times before going ahead.
---
-
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22772
@HyukjinKwon Sure. If there are more changes planned in another PR, then it
can take these changes in. Since this went to 2.4, i thought there may be some
urgency to fix the links.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22772
I think we better do some proof readings before doing multiple minor
followups.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22772
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/22772
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97567/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22772
**[Test build #97567 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97567/testReport)**
for PR 22772 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22772
**[Test build #97567 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97567/testReport)**
for PR 22772 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22772
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22772
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22772
cc @dongjoon-hyun @gatorsmile @xuanyuanking
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
14 matches
Mail list logo