[KYLIN-1396] minor bug in BigDecimalSerializer - avoidVerbose should be incremented each time when input scale is larger than given scale

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: v1.4.0
• Fix Version/s: v1.3.0, v1.4.0, v1.5.0
• Assignee: Edward Zhang
• Reporter: Edward Zhang
• Due: 05/Feb/16
• Created: 03/Feb/16 00:52
• Updated: 14/Mar/16 01:55
• Resolved: 03/Feb/16 06:42

Description

avoidVerbose is used for suppressing the warning message, but it should be incremented each time when input scale is larger than given scale

public void serialize(BigDecimal value, ByteBuffer out) {
if (value.scale() > type.getScale()) {
if (avoidVerbose++ % 10000 == 0)

{ logger.warn("value's scale has exceeded the " + type.getScale() + ", cut it off, to ensure encoded value do not exceed maxLength " + maxLength + " times:" + (avoidVerbose)); }

value = value.setScale(type.getScale(), BigDecimal.ROUND_HALF_EVEN);
}