Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/22913
Sounds good, thanks @javierluraschi !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user javierluraschi commented on the issue:
https://github.com/apache/spark/pull/22913
Yes, I believe that's the case. This change
https://github.com/apache/arrow/pull/2887 maps R bindings from `POSIXct` to
`timestamp` instead of `date`, looks like there is not need for this
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/22913
I'm a little against adding this because the Arrow Java Vectors used so far
were done to match the internal data of Spark, to keep things simple and avoid
lots of conversions on the Java side.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22913
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/22913
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98482/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22913
**[Test build #98482 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98482/testReport)**
for PR 22913 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22913
**[Test build #98482 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98482/testReport)**
for PR 22913 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22913
We map `POSIXct` to `TimestampType` in R. I think it makes more sense to
not support this. Historically Parquet also choses to better not support. For
instance, `UINT_*`.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22913
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/22913
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98455/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22913
**[Test build #98455 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98455/testReport)**
for PR 22913 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22913
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98454/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22913
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/22913
**[Test build #98454 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98454/testReport)**
for PR 22913 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22913
Also cc @ueshin
---
-
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/22913
**[Test build #98455 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98455/testReport)**
for PR 22913 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22913
**[Test build #98454 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98454/testReport)**
for PR 22913 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22913
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/22913
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98418/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22913
**[Test build #98418 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98418/testReport)**
for PR 22913 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22913
Can we add a test in `ArrowUtilsSuite.scala`?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22913
**[Test build #98418 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98418/testReport)**
for PR 22913 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22913
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22913
cc @BryanCutler
---
-
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/22913
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/22913
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/22913
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
27 matches
Mail list logo