[GitHub] [incubator-uniffle] jerqi commented on pull request #727: [#133][Netty] Netty Utils.

["jerqi (via GitHub)" <gi...@apache.org>]

jerqi commented on PR #727:
URL: https://github.com/apache/incubator-uniffle/pull/727#issuecomment-1470148475

   > > cc @jerqi Why does profile=spark2.3 need to set the version of netty separately? see https://github.com/apache/incubator-uniffle/blob/master/pom.xml
   > > ```
   > > <id>spark2.3</id>
   > >       <properties>
   > >         <scala.binary.version>2.11</scala.binary.version>
   > >         <spark.version>2.3.4</spark.version>
   > >         <client.type>2</client.type>
   > >         <jackson.version>2.9.0</jackson.version>
   > >         <netty.version>4.1.47.Final</netty.version>
   > > ```
   > 
   > The constructor of netty 4.1.47.Final's PooledByteBufAllocator is inconsistent. Can we use the same netty version? maybe there are more problems.
   
   It should be used for tests. If we change the version, the tests pass, it's ok to remove the version.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@uniffle.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@uniffle.apache.org
For additional commands, e-mail: issues-help@uniffle.apache.org