[GitHub] [hbase] YutSean commented on pull request #5352: HBASE-26780 HFileBlock.verifyOnDiskSizeMatchesHeader throw IOException: Passed in onDiskSizeWithHeader= A != 33

2023-09-06 Thread via GitHub


YutSean commented on PR #5352:
URL: https://github.com/apache/hbase/pull/5352#issuecomment-1709486972

   > How have you tested this? Could we write a unit test which demonstrates 
the intended behavior?
   As it is an unreproducible case, it is hard to simulate the error we met. 
The normal use case is already covered by our existing UTs. So that I did not 
write a UT. As for testing, we just pushed this change to our production 
environment. It works so far so 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: issues-unsubscr...@hbase.apache.org

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



[GitHub] [hbase] YutSean commented on pull request #5352: HBASE-26780 HFileBlock.verifyOnDiskSizeMatchesHeader throw IOException: Passed in onDiskSizeWithHeader= A != 33

2023-08-15 Thread via GitHub


YutSean commented on PR #5352:
URL: https://github.com/apache/hbase/pull/5352#issuecomment-1679851089

   The failed UT is unrelated to the change


-- 
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-unsubscr...@hbase.apache.org

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