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

[FLINK-24302][Connector/Pulsar] Drop the use of direct buffer memory for supporting Java 11. #27

Closed
wants to merge 1 commit into from

Conversation

syhily
Copy link
Contributor

@syhily syhily commented Feb 13, 2023

This is working in progress PR for testing if we have supported Java 11 by changing the Netty allocator type to heap instead of direct buffer memory.

@cbornet
Copy link
Contributor

cbornet commented May 27, 2023

Is this still needed now that Pulsar 2.11 client is used ?

@syhily
Copy link
Contributor Author

syhily commented Jul 18, 2023

Is this still needed now that Pulsar 2.11 client is used ?

Yep. The main issue isn't resolved after Pulsar has released the 3.0.0

@tisonkun
Copy link
Member

@syhily is there an issue or related PRs in the Pulsar side to describe why we have to do these tricks in the connector side?

@syhily
Copy link
Contributor Author

syhily commented Jul 18, 2023

@syhily is there an issue or related PRs in the Pulsar side to describe why we have to do these tricks in the connector side?

Yep. Here it is. apache/pulsar#17989

@syhily
Copy link
Contributor Author

syhily commented Jul 19, 2023

Close this PR for in favor of the #55.

@syhily syhily closed this Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants