yandrey321 commented on PR #10546: URL: https://github.com/apache/ozone/pull/10546#issuecomment-4801248579
> We were looking at this during the US community sync today and had a hard time following what is happening. Can you: > > * Get the markdown table to render correctly. > * Provide units for all measurements. For example: > > * We weren't sure whether `thr` was threads or throughput. If throughput it needs a unit. > * It's unclear what the unit on `CRC32 before/after` is. ops/sec? > * The stray `DN Latency` lines in the description need more context > * Specify what this will look like in a real cluster outside of the simulated environment. Updated description and provided summary for benchmark testing. Still need to evaluate the fix in the real cluster scenarious. -- 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]
