dioptre commented on PR #40729: URL: https://github.com/apache/arrow/pull/40729#issuecomment-2015306205
I was thinking the same. Perhaps we go for the scale first if you want to keep it in that format for the future and we can optimize later. If you are happy with that I can change denominator to scale and we can look at cache later. On Fri, Mar 22, 2024 at 06:32 Dominik Moritz ***@***.***> wrote: > I think the best way forward is to get this pull request in but make a new > issue that this pull request is attached to. We probably need to refactor > the whole decimal handling in ways that are orthogonal to this fix. How > does that sound @dioptre <https://github.com/dioptre>? > > — > Reply to this email directly, view it on GitHub > <https://github.com/apache/arrow/pull/40729#issuecomment-2015106723>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AAFZTGCLRZM7FC46J25DUFLYZQXFVAVCNFSM6AAAAABFCPNKICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJVGEYDMNZSGM> > . > You are receiving this because you were mentioned.Message ID: > ***@***.***> > -- 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]
