Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/1138
Another thought. The removed code is at plan time. Did the original code
have to open each file to retrieve schema? If so, does removing the code remove
that load? If so, then this change could be a huge performance improvement if
avoids the need to open every file in the Foreman.
Then, the the next question is: do we actually do anything with the
plan-time type information? Few files have that information. Given that, does
the planner actually use the information? Is this something we get for free
from Calcite? If we are not using the type information at plan time, then
clearly there is no harm in removing the code that retrieves the type
information.
---
