Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GH-36957]: [Java][C++] Set Protobuf to BUNDLED for backward compatibility #41860

Closed
wants to merge 1 commit into from

Conversation

jinchengchenghh
Copy link
Contributor

Rationale for this change

Fix Protobuf version compatible issue

What changes are included in this PR?

Protobuf dependency change from auto to bundled

Are these changes tested?

Current UT covered.

Are there any user-facing changes?

No

@jinchengchenghh jinchengchenghh requested a review from lidavidm as a code owner May 28, 2024 07:51
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}

In the case of PARQUET issues on JIRA the title also supports:

PARQUET-${JIRA_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

See also:

Copy link
Member

@lidavidm lidavidm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From your comment on the issue, it seems you actually ran into #39919. Can you use 15.0.2 or preferably the latest version of Arrow instead?

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting review Awaiting review labels May 28, 2024
@jinchengchenghh
Copy link
Contributor Author

Thanks! After I upgrade to 15.0.2, Jni freeBytes issue happens, change to version 16.0.0, it works well.

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.

2 participants