paul-rogers opened a new pull request #1796: DRILL-7261: Simplify Easy 
framework config for new scan
URL: https://github.com/apache/drill/pull/1796
 
 
   Most format plugins are created using the Easy format plugin. A recent
   change added support for the "row set" scan framework. After converting
   the text and log reader plugins, it became clear that the setup code
   could be made simpler.
   
   * Add the user name to the "file scan" framework.
   * Pass the file system, split and user name to the batch reader via
    the "schema negotiator" rather than via the constructor.
   * Create the traditional "scan batch" scan or the new row-set scan via
   functions instead of classes.
   * Add Easy config option and method to choose the kind of scan
   framework.
   * Add Easy config options for some newer options such as whether the
   plugin supports statistics.
   
   Tested by running all unit tests for the CSV reader which is based on
   the new framework, and by testing the converted log reader (that reader
   is not part of this commit.)

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to