lsyldliu commented on code in PR #20243:
URL: https://github.com/apache/flink/pull/20243#discussion_r923011831


##########
flink-table/flink-table-common/src/main/java/org/apache/flink/table/data/DecimalData.java:
##########
@@ -48,7 +48,7 @@ public final class DecimalData implements 
Comparable<DecimalData> {
     static final int MAX_INT_DIGITS = 9;
 
     /** Maximum number of decimal digits a Long can represent. (1e18 < 
Long.MaxValue < 1e19) */
-    static final int MAX_LONG_DIGITS = 18;
+    public static final int MAX_LONG_DIGITS = 18;

Review Comment:
   why not use `DecimalDataUtils.isByteArrayDecimal` instead of exposing it?



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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to