mdedetrich commented on PR #2409:
URL: https://github.com/apache/pekko/pull/2409#issuecomment-3606898178

   @raboof @pjfanning Apologies for missing this for the Pekko 2.0.0-M1 release 
train, I am still working on this but its taking longer as expected. I am 
actually currently implementing this work as a test in zstd-jni to make my life 
easier and also with the hope of getting feedback from the author that it is on 
the right track.
   
   Essentially the problem is that there isn't any real prior art for this, all 
of the current zstd streaming tests in zstd-jni either involve creating a 
direct `ByteBuffer` with the same size of the input data (which we obviously 
don't want to do in pekko-streams as these allocations are extremely expensive) 
or the input source is already a direct `ByteBuffer` (i.e. java nio).
   
   I am slightly annoying that I didn't make the 2.0.0-M1 release, but as long 
as we are happy with using a nightly in pekko-http to implement the http 
request/response zstd work in the end it shouldn't be that problematic. You 
could also argue its better this way, as pekko 2.0.0 milestone was primarily 
about updating all of the core libraries/jvm versions + removing deprecated 
code and bookkeeping and we can have 2.0.0-M2 be about major features.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to