Github user paul-rogers commented on a diff in the pull request:
    --- Diff: exec/vector/src/main/codegen/templates/ 
    @@ -514,6 +516,22 @@ public boolean isNull(int index){
        *   The equivalent Java primitive is '${minor.javaType!type.javaType}'
        * NB: this class is automatically generated from ValueVectorTypes.tdd 
using FreeMarker.
    +   * </p>
    +   * <h2>Contract</h2>
    +   * <p>
    +   *   Variable length vectors do not support random writes. All set 
methods must be called for with a monotonically increasing consecutive sequence 
of indexes.
    +   *   It is possible to trim the vector by setting the value count to be 
less than the number of values currently container in the vector with {@link 
#setValueCount(int)}, then
    +   *   the process of setting values starts with the index after the last 
    +   * </p>
    +   * <p>
    +   *   It is also possible to back track and set the value at an index 
earlier than the current index, however, the caller must assume that all data 
container after the last
    +   *   set index is corrupted.
    --- End diff --
    Maybe add "changing the index does not release memory after the index."


Reply via email to