Skip to content

Conversation

@xxlaykxx
Copy link

@xxlaykxx xxlaykxx commented Oct 3, 2025

Updated ComplexCopier to support ExtensionType - it contains two copy methods

public static void copy(FieldReader input, FieldWriter output) //for not breaking  existing logic

public static void copy(FieldReader input, FieldWriter output, ExtensionTypeWriterFactory extensionTypeWriterFactory)

Also updated ComplexCopier tests.
Closes apache/arrow-java#836.

ExtensionReader was added to support reading extension types from a complex vector.
It contains read(ExtensionHolder) method for reading to the holder. And readObject - for reading the value explicitly.

Closes apache/arrow-java#725.

also pr included fix for building jar for macos

xxlaykxx and others added 10 commits October 2, 2025 10:28
## What's Changed
ExtensionReader was added to support reading extension types from a
complex vector.
It contains **read(ExtensionHolder)** method for reading to the holder.
And **readObject** - for reading the value explicitly.

Closes apache#725.

(cherry picked from commit e6da71e871a3678a737a88dbe79491b4111496b4)
## What's Changed

Apply pre-commit since I forgot.

(cherry picked from commit e4f64269db0a08299fa25be491570be5ba71d623)
…e#837)

## What's Changed

Updated ComplexCopier to support ExtensionType - it contains two
**copy** methods
```
public static void copy(FieldReader input, FieldWriter output) //for not breaking  existing logic

public static void copy(FieldReader input, FieldWriter output, ExtensionTypeWriterFactory extensionTypeWriterFactory)
```
Also updated ComplexCopier tests.
Closes apache#836.

(cherry picked from commit 34060eb491a870f5ede5d30e007060b8310dc64f)
@github-actions
Copy link

github-actions bot commented Oct 3, 2025

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

@xxlaykxx xxlaykxx merged commit 45d8010 into dremio:dremio_26.1_18.1.0 Oct 3, 2025
8 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants