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/
---