paul-rogers commented on a change in pull request #2192:
URL: https://github.com/apache/drill/pull/2192#discussion_r605398703
##########
File path:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/scan/columns/ColumnsScanFramework.java
##########
@@ -103,7 +103,7 @@ protected void configure() {
}
@Override
- protected SchemaNegotiatorImpl newNegotiator() {
+ public SchemaNegotiatorImpl newNegotiator() {
Review comment:
As it turns out, cleaning up this kind of issue is one of the things
that the new version does.
There is also a race condition. The schema negotiator is passed into the
reader; it is not meant to be obtained elsewhere. In general, the framework
does a bunch of initialization, getting things ready. Then, each reader is
created, one at a time, passing in the schema negotiator. If we have code
trying to access it at some other time, then we'd have to review all that code
to see if new pattern works.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
