My comment is not connected to this change, but it would be good to change `ValueHolderHelper.getVarDecimalHolder` method to take `BigDecimal` instance instead of `String` since in all places, where it is used, the string is obtained from the `BigDecimal` and inside this method `BigDecimal` is created using this string.
[ Full content available at: https://github.com/apache/drill/pull/1481 ] This message was relayed via gitbox.apache.org for [email protected]
