Skip to content

Conversation

@xxlaykxx
Copy link

…6 (apache#36926)

When I used netty arrow memory 13.0.0 and netty 4.1.96.Final in Spark, the following error occurred, Because netty 4.1.96.Final version has revert some modifications, in order to ensure that netty arrow memory 13.0.0 works well with netty 4.1.96.Final version, I suggest making similar modifications here. 1.Compilation errors are as follows:
https://ci.appveyor.com/project/ApacheSoftwareFoundation/spark/builds/47657403 image

2.Some modifications have been reverted in netty 4.1.96.Final as follows: image image https://netty.io/news/2023/07/27/4-1-96-Final.html netty/netty#13510

Authored-by: panbingkun [email protected]

… 4.1.96 (apache#36926)

When I used `netty arrow memory 13.0.0` and `netty 4.1.96.Final` in Spark, the following error occurred,
Because `netty 4.1.96.Final` version has revert some modifications, in order to ensure that `netty arrow memory 13.0.0` works well with ``netty 4.1.96.Final`` version, I suggest making similar modifications here.
1.Compilation errors are as follows:
https://ci.appveyor.com/project/ApacheSoftwareFoundation/spark/builds/47657403
<img width="955" alt="image" src="https://github.com/apache/arrow/assets/15246973/e7ee2da9-97c0-474c-a62d-5821858e361f">

2.Some modifications have been reverted in `netty 4.1.96.Final` as follows:
<img width="884" alt="image" src="https://github.com/apache/arrow/assets/15246973/0226685a-cfa3-4b8b-b114-23ad8d027c05">
<img width="907" alt="image" src="https://github.com/apache/arrow/assets/15246973/a6ea21a0-8531-42b6-ab9d-25eaab1c7fde">
https://netty.io/news/2023/07/27/4-1-96-Final.html
netty/netty#13510
* Closes: apache#36928

Authored-by: panbingkun <[email protected]>
Signed-off-by: David Li <[email protected]>
@github-actions
Copy link

❌ GitHub issue apache#36928 could not be retrieved.

@xxlaykxx xxlaykxx merged commit 549a3ca into dremio:dremio_24.3_12.0 Jul 30, 2023
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.

2 participants