Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/1105#discussion_r172024863 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderRootExec.java --- @@ -231,7 +231,7 @@ public boolean innerNext() { } @VisibleForTesting - protected void createPartitioner() throws SchemaChangeException { + protected synchronized void createPartitioner() throws SchemaChangeException { --- End diff -- Why you need to synchronized`createParitioner`? It's being called from `innerNext` which is already synchronized.
---