sodonnel commented on PR #6613: URL: https://github.com/apache/ozone/pull/6613#issuecomment-3481813143
> Not sure about you question. Here is an example: > > * Client: opens the block and requests with 10KB. > Client: sends an onNext() request with 10KB. > Server: Since this is the first request, it opens the file. > Server: invokes a readChunk call to read 1MB (rounding up the chunk boundary) > Server: sends an onNext() response with 1MB data. After here, and the next 1MB read, the server will have released the handler thread and hence all the context it had including the open file. The default checksum size in Ozone is 16kb. So for a client reading into 4kb buffers, which is what I have seen happen, will result in many calls coming from the client for onNext, and the server having to re-establish context for each one. This is why I believe the performance with this suggestion will not be good. -- 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]
