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

    https://github.com/apache/drill/pull/984#discussion_r150096261
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/test/rowSet/RowSet.java ---
    @@ -85,8 +85,7 @@
        * new row set with the updated columns, then merge the new
        * and old row sets to create a new immutable row set.
        */
    -
    -  public interface RowSetWriter extends TupleWriter {
    +  interface RowSetWriter extends TupleWriter {
    --- End diff --
    
    IntelliJ gave a warning that the modifier is redundant. Also an interface 
nested inside another interface is public by default.
    
    https://beginnersbook.com/2016/03/nested-or-inner-interfaces-in-java/


---

Reply via email to