apex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ilganeli <...@git.apache.org>
Subject [GitHub] incubator-apex-core pull request: [APEXCORE-332][WIP] Improve part...
Date Fri, 27 May 2016 01:00:10 GMT
GitHub user ilganeli opened a pull request:


    [APEXCORE-332][WIP] Improve partitioning to support partitioning other than powers of

    * Updated the Partitioner to store partition definitions as Sets of Ranges which define
which values may be accepted by what ports
    * This substantially simplifies both the creation of new partitioning schemes (e.g. evenly
distributing values) and the logic around assigning values to partitions (no confusing mask,
simply a Range.contains() check)
    This is still a work in progress but given that it's touching a fairly intimate piece
of the core code it would be great to get some other eyes on this.

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

    $ git pull https://github.com/ilganeli/incubator-apex-core APEXCORE-332

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


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

    This closes #347
commit d95e83ea8d7317cf7faf5e32bd1244787e143084
Author: Ilya Ganelin <ilya.ganelin@capitalone.com>
Date:   2016-05-27T00:55:41Z

    Updated PartitionKeys to eliminate the use of mask. This class now defines a Set of Range
values which break up the range of possible values for a partitioning scheme. Some of the
updates to the reset of the dependent code have been made but this is still a work in progress.


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.

View raw message