fresh-borzoni commented on PR #175: URL: https://github.com/apache/fluss-rust/pull/175#issuecomment-3764885941
rust_decimal is capped at 28 digits and is typically faster than BigDecimal. One option is a hybrid design: a FlussDecimal wrapper that stores small decimals as rust_decimal::Decimal and falls back to BigDecimal for higher precision. That would only pay off if decimals are used heavily on hot paths, and it would noticeably increase implementation and maintenance complexity. For now, the BigDecimal approach is the right trade-off: prioritize correctness and wire-compatibility with Java. The hybrid optimization is something we can revisit later if profiling shows decimals are a real bottleneck. @leekeiabstraction @luoyuxia @Kelvinyu1117 PTAL 🙏 -- 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]
