Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19607
Thanks. I was just worried if I missed any discussion somewhere and wanted
to double check.
---
-
To unsubscribe, e-mail: re
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/19607
Ah, yes, I think so.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19607
Hm .. I see. but is this something we should fix though ideally? I am
asking this because I am checking R related codes now ..
---
---
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/19607
Maybe we need at least 2 external libraries like `dateutil` and `pytz` to
handle timezones properly, which Pandas uses, but I have no idea to handle
timezones out of Pandas properly for now.
---
-
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19607
Yup it should be separate. I meant to file another JIRA while we are here
if it is something we need to fix before we fix before forgetting. If
`df.collect()` is not meant to be fixed, I think I
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/19607
Unfortunately, `df.collect()` is out of scope of this pr. Its timestamp
values will respect python system timezone.
---
-
To unsu
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19607
Sorry, but does anyone remember how we are going to deal with
`df.collect()` in PySpark? R fix should be more like `df.collect()`. It should
be good to file a JIRA for `df.collect()` in PySpark
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19607
Yup, I was testing and trying to produce details. Let me describe this in
the JIRA, not here :D.
---
-
To unsubscribe, e-mai
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19607
LGTM, merging to master!
Let's fix the R timestamp issue in a new ticket.
---
-
To unsubscribe, e-mail: reviews-unsubs
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19607
Yup, I think we should take a look between POSIXct / POSIXlt in R and
timestamp within Spark too. Seems not respecting the session timezone in a
quick look.
---
--
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19607
I guess we should look at R to see if it should behavior similarly? WDYT
@HyukjinKwon ?
---
-
To unsubscribe, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84242/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #84242 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84242/testReport)**
for PR 19607 at commit
[`9200f38`](https://github.com/apache/spark/commit/9
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19607
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #84242 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84242/testReport)**
for PR 19607 at commit
[`9200f38`](https://github.com/apache/spark/commit/92
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84210/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #84210 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84210/testReport)**
for PR 19607 at commit
[`40a9735`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84205/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #84210 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84210/testReport)**
for PR 19607 at commit
[`40a9735`](https://github.com/apache/spark/commit/40
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19607
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #84205 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84205/testReport)**
for PR 19607 at commit
[`40a9735`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84204/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #84204 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84204/testReport)**
for PR 19607 at commit
[`f92eae3`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #84205 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84205/testReport)**
for PR 19607 at commit
[`40a9735`](https://github.com/apache/spark/commit/40
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #84204 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84204/testReport)**
for PR 19607 at commit
[`f92eae3`](https://github.com/apache/spark/commit/f9
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19607
> I tried to find out a workaround for old Pandas, but I haven't done yet.
I haven't looked at this closely yet but will definitely try to take a look
and help soon together. I would app
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19607
Yea, that was my proposal. If anything is blocked by this, I think we
should bump it up as, IMHO, technically the fixed version specification was not
yet released and published.
^ cc @c
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/19607
@BryanCutler I guess the oldest version of Pandas is `0.13.0` currently
according to #18403, cc @HyukjinKwon.
---
-
To unsubscri
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/19607
Hi @ueshin , what is the oldest version of Pandas that's required to
support and what exactly wasn't working with it?
---
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83295/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83295 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83295/testReport)**
for PR 19607 at commit
[`6872516`](https://github.com/apache/spark/commit/6
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/19607
@BryanCutler I'm fixing the behavior of `toPandas()` and pandas udfs as we
discussed in #18664 but I guess we still need to support old Pandas as well.
I tried to find out a workaround for old Pan
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83295 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83295/testReport)**
for PR 19607 at commit
[`6872516`](https://github.com/apache/spark/commit/68
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83294/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83286/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83286 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83286/testReport)**
for PR 19607 at commit
[`b1436b8`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83286 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83286/testReport)**
for PR 19607 at commit
[`b1436b8`](https://github.com/apache/spark/commit/b1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83280/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83280 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83280/testReport)**
for PR 19607 at commit
[`ee1a1c8`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83280 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83280/testReport)**
for PR 19607 at commit
[`ee1a1c8`](https://github.com/apache/spark/commit/ee
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83211/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83211 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83211/testReport)**
for PR 19607 at commit
[`e28fc87`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83212/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83212 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83212/testReport)**
for PR 19607 at commit
[`5c08ecf`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83211 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83211/testReport)**
for PR 19607 at commit
[`e28fc87`](https://github.com/apache/spark/commit/e2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83212 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83212/testReport)**
for PR 19607 at commit
[`5c08ecf`](https://github.com/apache/spark/commit/5c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83207/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83207 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83207/testReport)**
for PR 19607 at commit
[`5c08ecf`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83207 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83207/testReport)**
for PR 19607 at commit
[`5c08ecf`](https://github.com/apache/spark/commit/5c
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/19607
Jenkins, retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83205/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19607
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83205 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83205/testReport)**
for PR 19607 at commit
[`5c08ecf`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19607
**[Test build #83205 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83205/testReport)**
for PR 19607 at commit
[`5c08ecf`](https://github.com/apache/spark/commit/5c
65 matches
Mail list logo