cgivre commented on pull request #2427: URL: https://github.com/apache/drill/pull/2427#issuecomment-1013768959
> @cgivre see latest comment on https://issues.apache.org/jira/browse/DRILL-8095 - not loading the styles breaks existing Drill tests - so I abandoned that bit of the change Roger that.... Sorry I didn't see that. We don't have to include it in this PR, but if you can recommend a better way of inferring the data type that would be more performant, I'm all ears. I did it using the Cell style because that seemed to make the most sense. Thanks! -- 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
