klion26 commented on PR #9689: URL: https://github.com/apache/arrow-rs/pull/9689#issuecomment-4312146633
Based on the new benchmark result, I'll undo the change for `decimal to integer` to see if there regression was come from the benchmark it self, the `undo change` is mainly for test, will revert the last commit after get the benchmark result The new benchmark result shows something, - Some cases(`cast decimal64 to int8`/`cast decimal64 to uint8`/`cast invalid float32 to decimal32(9, 2)`) that were regression before are gone this time - There are some cases(`cast decimal32 to int8`/`cast decimal32 to uint8`) regression which did not show before - The regression time is less than before. -- 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]
