Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23166
thanks @redsanket @squito, committed master, 2.4, and 2.3.2
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99401/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
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/23166
**[Test build #99401 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99401/testReport)**
for PR 23166 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
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/23166
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99399/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23166
**[Test build #99399 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99399/testReport)**
for PR 23166 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99394/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
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/23166
**[Test build #99394 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99394/testReport)**
for PR 23166 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/23166
cc @HyukjinKwon @ueshin
---
-
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/23166
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/23166
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/23166
**[Test build #99401 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99401/testReport)**
for PR 23166 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23166
**[Test build #99399 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99399/testReport)**
for PR 23166 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
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/23166
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user redsanket commented on the issue:
https://github.com/apache/spark/pull/23166
@squito yeah looking at the code we could maintain a soft reference so that
we dont have to keep requesting it from the disk. It is definitely an
optimization
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23166
**[Test build #99394 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99394/testReport)**
for PR 23166 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
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/23166
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user squito commented on the issue:
https://github.com/apache/spark/pull/23166
> The only question I have is does this cause more memory usage on the
driver because it has a reference to that broadcast value or is something else
already holding on to it?
yeah good
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23166
Yeah certainly seems like a good idea. The only question I have is does
this cause more memory usage on the driver because it has a reference to that
broadcast value or is something else already
Github user squito commented on the issue:
https://github.com/apache/spark/pull/23166
maybe a dumb question -- couldn't you just add
```python
self._value = value
```
[inside `__init__()` when on the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
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/23166
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/23166
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99387/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23166
**[Test build #99387 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99387/testReport)**
for PR 23166 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23166
**[Test build #99387 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99387/testReport)**
for PR 23166 at commit
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23166
cc @squito
---
-
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/23166
ok to test
---
-
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/23166
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/23166
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/23166
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
35 matches
Mail list logo
