Github user fhueske commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4668#discussion_r138875420
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/operators/util/BitSet.java 
---
    @@ -56,8 +56,7 @@ public void setMemorySegment(MemorySegment memorySegment, 
int offset) {
         * @param index - position
         */
        public void set(int index) {
    -           Preconditions.checkArgument(index < bitLength && index >= 0, 
    -                   String.format("Input Index[%d] is larger than BitSet 
available size[%d].", index, bitLength));
    +           Preconditions.checkArgument(index < bitLength && index >= 0);
    --- End diff --
    
    Can be changed to 
    
    ```
    Preconditions.checkArgument(index < bitLength && index >= 0, 
      "Input Index[%d] is larger than BitSet available size[%d].", index, 
bitLength);
    ``` 
    to perserve the error message.


---

Reply via email to