seddonm1 commented on pull request #797:
URL: https://github.com/apache/arrow-datafusion/pull/797#issuecomment-892218323


   Thank you for the review @alamb.
   
   I have refactored to reduce repetition (by reusing some existing methods) 
but still have the ambiguous column problem outstanding.
   
   I would like to know your thoughts on merging this one first and then I will 
raise another issue and make a PR to address the ambiguity (which is an issue 
on master already).


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

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to