Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22873
thanks, merging to master/2.4!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/22873
LGTM. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22873
any comments on this?
---
-
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/22873
**[Test build #98196 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98196/testReport)**
for PR 22873 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22873
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98196/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22873
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/22873
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/22873
**[Test build #98196 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98196/testReport)**
for PR 22873 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22873
@viirya I think we can remove it as it is used nowhere. The point is: even
if we remove it, we will remove it only on 3.0, so for 2.4 we should anyway fix
this IMO.
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22873
Are we considering to remove it? If no, as `save` now calls `SaveLoadV2_0`,
it seems good to update this variable.
---
-
To
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22873
cc @cloud-fan @huaxingao @mengxr @WeichenXu123 @viirya
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
11 matches
Mail list logo