Github user srowen commented on the issue:
https://github.com/apache/spark/pull/23247
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@
Github user wangjiaochun commented on the issue:
https://github.com/apache/spark/pull/23247
ok! thanks, I will focus on commit bug later.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additio
Github user wangjiaochun commented on the issue:
https://github.com/apache/spark/pull/23247
> @wangjiaochun, I think you better stop fixing trivial stuff in each PR.
Those stuff can be fixed when the codes around here is fixed, or let other
people fix it later.
ok! thanks, I
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23247
**[Test build #4456 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4456/testReport)**
for PR 23247 at commit
[`d5aea03`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23247
**[Test build #4456 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4456/testReport)**
for PR 23247 at commit
[`d5aea03`](https://github.com/apache/spark/commit/d
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/23247
These aren't worth the time it takes us to review them and merge them,
honestly. Little cleanup can be OK if it makes an appreciable difference in
speed or readability, and if you can find many insta
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23247
@wangjiaochun, I think you better stop fixing trivial stuff in each PR.
Those stuff can be fixed when the codes around here is fixed, or let other
people fix it later.
---
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23247
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/23247
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/23247
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
10 matches
Mail list logo