Github user vrozov commented on the issue:
https://github.com/apache/drill/pull/1144
It is not clear why get/set Byte/Char/Short/Int/Long/Float/Double do not
delegate to UDLE, while get/set Bytes delegates to UDLE and relies on netty
'AbstractByteBuf` for bounds checking. IMO, it will be good to have the
behavior consistent for all methods.
In many cases including `VariableLengthVectors`, there is no need to rely
on UDLE boundary checking as a caller already provides or can provide a
guarantee that an index is within a buffer boundaries. In those cases, boundary
check becomes an extra cost. IMO, it will be good to have a consistent behavior
with ability to enable bounds checking for debugging.
---
