You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2018/10/19 00:49:45 UTC

[GitHub] milleruntime opened a new pull request #712: Cleanup MapReduce package and API

milleruntime opened a new pull request #712: Cleanup MapReduce package and API
URL: https://github.com/apache/accumulo/pull/712
 
 
   Please see 4 separate commits for each step so far.  This is still a work in progress but I was looking for feedback before implementing each new MapReduce class.  
   
   The idea for the new API was to have each Accumulo*Format be required to only call one method:
   public static void setInfo(Job job, InputInfo info);
   
   The only object required with the Job would be a getter interface object that would have a builder to go along (see inputInfo and InputInfoImpl in 886db213ee7a5919d6828903adeb315dd65fa508).  This would only expose these methods and the getter interface object.  Everything else would stay under the hood allowing us to refactor and clean up the internals as we see fit.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services