Skip to content

[SPARK-21472][SQL][FOLLOW-UP] Introduce ArrowColumnVector as a reader for Arrow vectors.#18701

Closed
ueshin wants to merge 1 commit intoapache:masterfrom
ueshin:issues/SPARK-21472_fup1
Closed

[SPARK-21472][SQL][FOLLOW-UP] Introduce ArrowColumnVector as a reader for Arrow vectors.#18701
ueshin wants to merge 1 commit intoapache:masterfrom
ueshin:issues/SPARK-21472_fup1

Conversation

@ueshin
Copy link
Copy Markdown
Member

@ueshin ueshin commented Jul 21, 2017

What changes were proposed in this pull request?

This is a follow-up of #18680.

In some environment, a compile error happens saying:

.../sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ArrowColumnVector.java:243:
error: not found: type Array
  public void loadBytes(Array array) {
                        ^

This pr fixes it.

How was this patch tested?

Existing tests.

@viirya
Copy link
Copy Markdown
Member

viirya commented Jul 21, 2017

LGTM

@SparkQA
Copy link
Copy Markdown

SparkQA commented Jul 21, 2017

Test build #79831 has finished for PR 18701 at commit 55766b5.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@cloud-fan
Copy link
Copy Markdown
Contributor

LGTM, merging to master!

@asfgit asfgit closed this in 2f14684 Jul 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants