Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/22455
LGTM. merged to master
---
-
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/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97984/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
**[Test build #97984 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97984/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #97984 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97984/testReport)**
for PR 22455 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #4388 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4388/testReport)**
for PR 22455 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #4388 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4388/testReport)**
for PR 22455 at commit
Github user adrian555 commented on the issue:
https://github.com/apache/spark/pull/22455
Test was terminated as "Build execution time has reached the maximum
allowed time for your plan (90 minutes)."
Try to close and reopen the PR to see how the retest goes.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97936/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
**[Test build #97936 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97936/testReport)**
for PR 22455 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #97936 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97936/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
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/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97551/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
**[Test build #97551 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97551/testReport)**
for PR 22455 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #97551 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97551/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
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/22455
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/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97368/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #97368 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97368/testReport)**
for PR 22455 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #97368 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97368/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
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/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96906/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #96906 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96906/testReport)**
for PR 22455 at commit
Github user adrian555 commented on the issue:
https://github.com/apache/spark/pull/22455
> I'm not sure about the new changes. IMO we are mixing S3 and S4
convention here as per @adrian555.
The implemented code is still conforming to S4 object standard that
`show()` is the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #96906 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96906/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96645/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
**[Test build #96645 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96645/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96647/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #96647 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96647/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
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/22455
**[Test build #96647 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96647/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
**[Test build #96645 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96645/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user falaki commented on the issue:
https://github.com/apache/spark/pull/22455
@adrian555 yes, that looks good. Thank you!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user adrian555 commented on the issue:
https://github.com/apache/spark/pull/22455
Thanks @falaki. We are taking out the `print` method out so the requirement
seems clear now. Yes, for Jupyter with IRkernel, two options
`jupyter.rich_display` and jupyter.base_display_func` are
Github user falaki commented on the issue:
https://github.com/apache/spark/pull/22455
@adrian555 These all great points. My high-level was enabling other
platforms (e.g., Jupyter) to plugin more advanced (custom) functions for
displaying SparkDataFrame. If the framework does not set
Github user adrian555 commented on the issue:
https://github.com/apache/spark/pull/22455
Another flavor will be this
```r
print.SparkDataFrame <- function(x, ...) {
if ((length(list(...)) > 0) &&
identical(sparkR.conf("spark.sql.repl.eagerEval.enabled", "false")[[1]],
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96569/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
**[Test build #96569 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96569/testReport)**
for PR 22455 at commit
Github user adrian555 commented on the issue:
https://github.com/apache/spark/pull/22455
@falaki trying to understand what you were asking for. I know that `print`
takes some arguments like `digit` and `quote` that can further format the
output. `print` is for S3 object and `show` is
Github user falaki commented on the issue:
https://github.com/apache/spark/pull/22455
@adrian555 thanks for submitting this. Can we have a config to set the
default `print` function in eager mode. It can default to `show`, but I can
imagine it is useful to make it configurable. For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
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/22455
**[Test build #96569 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96569/testReport)**
for PR 22455 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/22455
> Looks like the test was terminated in the middle, not likely related to
the any code change. Could someone please ask for retest? Thanks.
appveyor one? you need to close and re-open
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22455
Great feature! cc @falaki
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96515/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
**[Test build #96515 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96515/testReport)**
for PR 22455 at commit
Github user adrian555 commented on the issue:
https://github.com/apache/spark/pull/22455
Looks like the test was terminated in the middle, not likely related to the
any code change. Could someone please ask for retest? Thanks.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
**[Test build #96515 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96515/testReport)**
for PR 22455 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22455
@adrian555 The changes look fine to me. Thank you.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96450/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #96450 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96450/testReport)**
for PR 22455 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #96450 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96450/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22455
Let's also update the doc of `REPL_EAGER_EVAL_ENABLED` in `SQLConf`. After
this patch, eager evaluation is not only supported in PySpark.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96387/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #96387 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96387/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
**[Test build #96387 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96387/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
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/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96383/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
**[Test build #96383 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96383/testReport)**
for PR 22455 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #96383 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96383/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
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/22455
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/22455
**[Test build #96381 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96381/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96381/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #96381 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96381/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
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/22455
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/22455
**[Test build #96203 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96203/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96203/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22455
**[Test build #96203 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96203/testReport)**
for PR 22455 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22455
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/22455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
100 matches
Mail list logo