pkuwm opened a new issue #910: Unit test TestZNRecordSerializeWriteSizeLimit fails URL: https://github.com/apache/helix/issues/910 Test log: ``` testZNRecordSerializerWriteSizeLimit(org.apache.helix.zookeeper.datamodel.serializer.TestZNRecordSerializeWriteSizeLimit) Time elapsed: 0.123 sec <<< FAILURE! org.apache.helix.zookeeper.zkclient.exception.ZkMarshallingError: Data size: 5445 is greater than 2000 bytes, is compressed: true, ZNRecord.id: record at org.apache.helix.zookeeper.datamodel.serializer.ZNRecordSerializer.serialize(ZNRecordSerializer.java:110) at org.apache.helix.zookeeper.datamodel.serializer.TestZNRecordSerializeWriteSizeLimit.verifyAutoCompression(TestZNRecordSerializeWriteSizeLimit.java:151) at org.apache.helix.zookeeper.datamodel.serializer.TestZNRecordSerializeWriteSizeLimit.testZNRecordSerializerWriteSizeLimit(TestZNRecordSerializeWriteSizeLimit.java:125) Results : Failed tests: TestZNRecordSerializeWriteSizeLimit.testAutoCompressionDisabled:64->verifyAutoCompression:151 » ZkMarshalling TestZNRecordSerializeWriteSizeLimit.testZNRecordSerializerWriteSizeLimit:125->verifyAutoCompression:151 » ZkMarshalling Tests run: 2, Failures: 2, Errors: 0, Skipped: 0 ``` Reason: There was a exception type change in znrecord serializers: ZkClientException -> ZkMarshallingError. So the exception types don't match in the unit test. Solution: Change the exception type in the unit test.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@helix.apache.org For additional commands, e-mail: reviews-h...@helix.apache.org
