paul-rogers opened a new pull request #1676: DRILL-7074: Scan framework fixes and enhancements URL: https://github.com/apache/drill/pull/1676 This PR is a roll-up of a number of fixes and enhancements to the new “scan framework” that resulted from the effort to migrate the CSV (compliant text) reader to the new framework. To keep reviews manageable, this PR includes just the scan framework changes. Since this framework is not yet used in any “production” code, this part of the review should be relatively easy. A separate PR will provide the reposted CSV reader. Changes include: * More documentation * Improved support for the “schema only” first batch that (it seems) downstream operators need. * Support for the original behavior of the wildcard with partitioned columns, and the proposed behavior identified in DRILL-7055. Restores behavior previously removed. An option switches between the two behaviors. * More permissive combinations of the `columns` column and the wildcard. (Turns out that current behavior ,as show, by unit tests, is pretty varied.) * Several renames and refactorings * Fixed a bug in how the projection set is passed to the result set loader. * Multiple new or revised unit 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
