sachouche commented on issue #1445: DRILL-6706: fixed null pointer exception in 
HashJoin
URL: https://github.com/apache/drill/pull/1445#issuecomment-416340872
 
 
   I think the behavior introduced in DRILL-4264 needs to be fixed:
   - Improve the metadata to flag a column as missing (no back-tick business)
   - All readers should apply the same fix
   - This will give an opportunity to upstream operators to provide explicit 
handling (e.g., Filter)
   
   @ilooner, ideally the RecordBatchSizer should be using SchemaPath as keys to 
handle nested column names; I guess the Planner is injecting a project step to 
disambiguate such use-cases.
   
   For now, I let you guys decide; we either:
   a) Accept this temporary workaround and create a new JIRA to implement the 
right fix
   b) Fix the Parquet reader (remove the back-tick) with the side-effect that 
the schema-on-change behavior for filter based queries is broken
   c) Implement the right fix for DRILL-4264 as discussed above

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to