accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] milleruntime opened a new pull request #712: Cleanup MapReduce package and API
Date Fri, 19 Oct 2018 00:49:45 GMT
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

Mime
View raw message