You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug, including details regarding any error messages, version, and platform.
In benchmarking pipelines we see that arrow-compute-scalar-round-benchmark fails with code 6, emitting Invalid: Rounding to -5 digits is out of range for type int16 on stderr. The first commit that produces this error was 7c8f398. It seems to be strongly related :).
Component(s)
Archery, Benchmarking
The text was updated successfully, but these errors were encountered:
pitrou
changed the title
arrow-compute-scalar-round-benchmark fails with Rounding to -5 digits is out of range for type int16
[C++] arrow-compute-scalar-round-benchmark fails with Rounding to -5 digits is out of range for type int16Aug 10, 2023
The easiest fix I can think of is to remove the benchmark for uint8, int8, uint16 and int16 types. I think the 32 and 64 bits are good enough to represent integers since they are the most commonly used types. Does it sound good to you? @pitrou
### Rationale for this change
The valid range of 8 and 16 bit integers is too small, which causes failures on `round` benchmarks.
### What changes are included in this PR?
`Round` benchmarks for <32 bit integers are removed. Arithmetic on small int widths is probably uncommon.
### Are these changes tested?
The benchmark is now running successfully on my machine.
### Are there any user-facing changes?
No.
* Closes: #37106
Authored-by: Jin Shang <shangjin1997@gmail.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
…pache#37109)
### Rationale for this change
The valid range of 8 and 16 bit integers is too small, which causes failures on `round` benchmarks.
### What changes are included in this PR?
`Round` benchmarks for <32 bit integers are removed. Arithmetic on small int widths is probably uncommon.
### Are these changes tested?
The benchmark is now running successfully on my machine.
### Are there any user-facing changes?
No.
* Closes: apache#37106
Authored-by: Jin Shang <shangjin1997@gmail.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
Describe the bug, including details regarding any error messages, version, and platform.
In benchmarking pipelines we see that
arrow-compute-scalar-round-benchmark
fails with code 6, emittingInvalid: Rounding to -5 digits is out of range for type int16
on stderr. The first commit that produces this error was 7c8f398. It seems to be strongly related :).Component(s)
Archery, Benchmarking
The text was updated successfully, but these errors were encountered: