accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From melrief <...@git.apache.org>
Subject [GitHub] accumulo pull request #145: ACCUMULO-4376 add KeyBuilder
Date Thu, 01 Sep 2016 00:00:53 GMT
GitHub user melrief opened a pull request:

    https://github.com/apache/accumulo/pull/145

    ACCUMULO-4376 add KeyBuilder

    Adds the KeyBuilder as proposed in ACCUMULO-4376. The PR addes a `KeyBuilder` which can
be used to add build `Key`s in a mutable way. For the design, I took inspiration from the
Step Builder design pattern as described in [Vladimir Stankovic blog](http://www.svlada.com/step-builder-pattern/)
to try to give some rules to the way `Key`s are built. I also added a new constructor to `Key`
that accepts all the parameters, including `deleted` similar to to the `Key.init` method,
that is used to build `Key`s from the fields.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/melrief/accumulo ACCUMULO-4376

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/accumulo/pull/145.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #145
    
----
commit 4ff5f0cfc1f7f890e68736a35de86925fccddb62
Author: Mario Pastorelli <mario.pastorelli@teralytics.ch>
Date:   2016-08-31T23:32:40Z

    ACCUMULO-4376 add KeyBuilder

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message