pkuwm commented on pull request #976:
URL: https://github.com/apache/helix/pull/976#issuecomment-634441483


   This PR is ready to be merged, approved by @narendly 
   
   ```
   In TestZNRecordSizeLimit, the assertion message should indicate the data is 
smaller than 1 MB threshold. HelixException should be ZkMarshallingError 
because the exception type is changed in ZNRecord serializer.
   ```


----------------------------------------------------------------
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.

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