paul-rogers commented on PR #2615: URL: https://github.com/apache/drill/pull/2615#issuecomment-1200668136
Jumping in a bit late to this one. It is generally a good idea to include unit tests with all changes. Can we add such tests in a subsequent PR? Since implicit columns affect wildcard expansion, it would be good to include unit tests to ensure things work as intended. Such tests can be added to, or based upon, the existing projection tests. -- 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: dev-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
