szetszwo commented on PR #6613: URL: https://github.com/apache/ozone/pull/6613#issuecomment-3482102886
> ** The point of the checksum change from 1MB -> 16kb by default was to allow for smaller reads, so we don't want to constrain it to be required to return 1MB at a time. ** If small is required, we could round up the checksum boundary. Updated below: - 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 10KB Server: sends an onNext() response with 16KB data (rounding up the chunk boundary). - Client: requests 20KB but the buffer only has 6KB left Client: sends an onNext() request with 14KB. Server: invokes a readChunk call to read 14KB. Server: sends an onNext() response with 16KB data (rounding up the chunk boundary). - The 50MB case is the same as before. -- 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]
