[SAMZA-2416] Adding null-check before incrementing metrics for bytesSerialized

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.4
• Assignee: Unassigned
• Reporter: Rayman
• Created: 10/Dec/19 21:56
• Updated: 28/Feb/20 19:24
• Resolved: 28/Feb/20 19:24

Description

Adding null-check before incrementing metrics for bytesSerialized

Symptom: User serde-serialization logic can choose to serialize non-null objects to a null, which implies delete the value. In this case, current code tries to increment the metrics using the size of the null and fails with an NPE.

Cause: Current code tries to increment the bytesSerialized metrics using the size of the null and fails with an NPE.

Fix: Added logic to skip metric value update if serde returns null.

Issue Links

• links to: GitHub Pull Request #1234