From notifications-return-45673-archive-asf-public=cust-asf.ponee.io@accumulo.apache.org Fri Oct 19 02:49:47 2018 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id 466D6180674 for ; Fri, 19 Oct 2018 02:49:47 +0200 (CEST) Received: (qmail 95773 invoked by uid 500); 19 Oct 2018 00:49:46 -0000 Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: jira@apache.org Delivered-To: mailing list notifications@accumulo.apache.org Received: (qmail 95759 invoked by uid 99); 19 Oct 2018 00:49:46 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 19 Oct 2018 00:49:46 +0000 From: GitBox To: notifications@accumulo.apache.org Subject: [GitHub] milleruntime opened a new pull request #712: Cleanup MapReduce package and API Message-ID: <153991018549.31460.17341874947016909286.gitbox@gitbox.apache.org> Date: Fri, 19 Oct 2018 00:49:45 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit 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