Skip to content

[C++][ORC] Sync implementation from li-arrow feature/orc-schema-manifest#24

Closed
cbb330 wants to merge 0 commit intofeature/orc-adapterfrom
feature/orc-schema-manifest
Closed

[C++][ORC] Sync implementation from li-arrow feature/orc-schema-manifest#24
cbb330 wants to merge 0 commit intofeature/orc-adapterfrom
feature/orc-schema-manifest

Conversation

@cbb330
Copy link
Owner

@cbb330 cbb330 commented Feb 25, 2026

Copy latest changes to adapter.cc, adapter.h, and adapter_test.cc from the li-arrow repository's feature/orc-schema-manifest branch.

Thanks for opening a pull request!

If this is your first pull request you can find detailed information on how to contribute here:

Please remove this line and the above text before creating your pull request.

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

This PR includes breaking changes to public APIs. (If there are any breaking changes to public APIs, please explain which changes are breaking. If not, you can remove this.)

This PR contains a "Critical Fix". (If the changes fix either (a) a security vulnerability, (b) a bug that caused incorrect or invalid data to be produced, or (c) a bug that causes a crash (even when the API contract is upheld), please provide explanation. If not, you can remove this.)

@github-actions
Copy link

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:

@cbb330 cbb330 force-pushed the feature/orc-schema-manifest branch from 81d234f to 42a0f8f Compare February 25, 2026 22:35
@cbb330 cbb330 closed this Feb 25, 2026
@cbb330 cbb330 force-pushed the feature/orc-schema-manifest branch from 42a0f8f to 5c50f0d Compare February 25, 2026 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant