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