accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rweeks <>
Subject [GitHub] accumulo pull request: Implementation of SeekingFilter, CfCqSliceF...
Date Thu, 13 Aug 2015 17:46:16 GMT
GitHub user rweeks opened a pull request:

    Implementation of SeekingFilter, CfCqSliceFilter and CfCqSliceSeekingFilter

    OK let's try this again. Right bug id this time.
    Implementation for ACCUMULO-3961 including a filter which behaves similarly to the existing
ColumnSliceFilter but can seek to the next potential match instead of explicitly rejecting
every KV pair that falls outside the slice. Probably faster for small slices on large rows.
    Christopher switched the fix version to 1.8, I guess because adding new functionality
should go into a minor release. So I'm targeting this against the master branch, because it
doesn't look like a 1.8 branch exists yet.
    I know you guys require this to be rebased into a single commit. Is that something I can
do on my side (I'm concerned about rebasing after I've pushed this to a public repo) or is
that something you can do when you receive the pull request?

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

    $ git pull ACCUMULO-3961

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 #42
commit a0c8d588109e2d8d0b3bdf862ec1d3e1e48c50e7
Author: Russ Weeks <>
Date:   2015-07-23T03:53:01Z

    Initial commit of CfCqSlice code. Renamed files but Java class names are messed up.

commit 9cc2c28251d445c86d7006bf1237ea2fd096b4e1
Author: Russ Weeks <>
Date:   2015-07-23T04:23:12Z

    Removing cyclic dependency on MiniAccumuloCluster by moving tests to test module

commit 1100ba49ee09c97aadbe80429f54a13519a98552
Author: Russ Weeks <>
Date:   2015-07-23T05:48:12Z

    Reformatted and added comment to SeekingFilter

commit e5ea222fa07ab8bb7211a12c87aa127146874f81
Author: Russ Weeks <>
Date:   2015-07-24T00:20:08Z

    Slice tests passing

commit f579acc4e60b7aa5cf36c59fad51ea36347204f0
Author: Russ Weeks <>
Date:   2015-08-13T17:21:31Z

    Added class-level javadoc, fixed problem with OptionsDescriber

commit dc2655622f71676012cf849a849e2e89fd936d26
Author: Russ Weeks <>
Date:   2015-08-13T17:44:38Z

    tweaking the javadoc.


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 or file a JIRA ticket
with INFRA.

View raw message