[GitHub] vvysotskyi commented on issue #1445: DRILL-6706: fixed null pointer exception in HashJoin

2018-08-28 Thread GitBox
vvysotskyi commented on issue #1445: DRILL-6706: fixed null pointer exception 
in HashJoin
URL: https://github.com/apache/drill/pull/1445#issuecomment-416532411
 
 
   @ilooner, I don't have any objection regarding this PR. If this bug is 
critical, it may be merged and DRILL-6714 with more proper fix may be done 
later.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] vvysotskyi commented on issue #1445: DRILL-6706: fixed null pointer exception in HashJoin

2018-08-27 Thread GitBox
vvysotskyi commented on issue #1445: DRILL-6706: fixed null pointer exception 
in HashJoin
URL: https://github.com/apache/drill/pull/1445#issuecomment-416167363
 
 
   @sachouche, I think more proper solution is to store `SchemaPath` instances 
or their string representations in `columnSizes` map to avoid ambiguities. 
Also, logic in `getComplexColumn` should be rewritten (all this code was added 
after DRILL-4264, but this logic breaks changes made in DRILL-4264).


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:
us...@infra.apache.org


With regards,
Apache Git Services