Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20239
Thanks for everyone for review!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20239
Merged to master and branch-2.3.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20239
@BryanCutler Any comments on this?
---
-
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/20239
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86048/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20239
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/20239
**[Test build #86048 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86048/testReport)**
for PR 20239 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20239
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86043/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20239
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/20239
**[Test build #86043 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86043/testReport)**
for PR 20239 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20239
**[Test build #86048 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86048/testReport)**
for PR 20239 at commit
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20239
`MapVector` is still used in Arrow internal code but it should not be
returned to user directly.
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20239
@ueshin and @BryanCutler I took another look and the class `StructAccessor`
defined in `ArrowColumnVector` never gets used for `getStruct`.
`ArrowColumnVector.getStruct()` method just calls
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20239
**[Test build #86043 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86043/testReport)**
for PR 20239 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20239
Btw, I don't mean to block this pr but why does only `MapVector` have
`Nullable` version, just out of curiosity.
---
-
To
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20239
I'm not sure we can change to `NullableMapVector` and I'm just worrying
whether the `MapVector` is never happened here.
---
-
To
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20239
@BryanCutler Yes there is no error currently. This should make the code
cleaner though.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20239
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85989/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20239
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/20239
**[Test build #85989 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85989/testReport)**
for PR 20239 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20239
**[Test build #85989 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85989/testReport)**
for PR 20239 at commit
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20239
@BryanCutler I think this comes up in the Arrow sync yesterday
---
-
To unsubscribe, e-mail:
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20239
cc @BryanCutler @ueshin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
22 matches
Mail list logo