Remove duplicate bound check in the function shift

[HaoYang670]

Signed-off-by: remzi 13716567376yh@gmail.com

Which issue does this PR close?

Closes #1408.

Rationale for this change

Replace clamp(-offset, 0, value_len) by clamp_min(-offset, 0). Because we have compared offset and value_len before.

Are there any user-facing changes?

No

[codecov-commenter]

Codecov Report

Merging #1409 (42858fc) into master (f19d1ed) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1409   +/-   ##
=======================================
  Coverage   83.16%   83.16%           
=======================================
  Files         182      182           
  Lines       53394    53394           
=======================================
+ Hits        44405    44406    +1     
+ Misses       8989     8988    -1     

Impacted Files	Coverage Δ
arrow/src/compute/kernels/window.rs	100.00% <100.00%> (ø)
arrow/src/datatypes/field.rs	53.79% <0.00%> (-0.31%)	⬇️
parquet/src/encodings/encoding.rs	93.71% <0.00%> (+0.19%)	⬆️
parquet_derive/src/parquet_field.rs	66.43% <0.00%> (+0.22%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f19d1ed...42858fc. Read the comment docs.

[sunchao]

Merged, thanks!