GitHub user rweeks opened a pull request:
https://github.com/apache/accumulo/pull/42
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 https://github.com/phemisystems/accumulo ACCUMULO-3961
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/accumulo/pull/42.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 #42
----
commit a0c8d588109e2d8d0b3bdf862ec1d3e1e48c50e7
Author: Russ Weeks <[email protected]>
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 <[email protected]>
Date: 2015-07-23T04:23:12Z
Removing cyclic dependency on MiniAccumuloCluster by moving tests to test
module
commit 1100ba49ee09c97aadbe80429f54a13519a98552
Author: Russ Weeks <[email protected]>
Date: 2015-07-23T05:48:12Z
Reformatted and added comment to SeekingFilter
commit e5ea222fa07ab8bb7211a12c87aa127146874f81
Author: Russ Weeks <[email protected]>
Date: 2015-07-24T00:20:08Z
Slice tests passing
commit f579acc4e60b7aa5cf36c59fad51ea36347204f0
Author: Russ Weeks <[email protected]>
Date: 2015-08-13T17:21:31Z
Added class-level javadoc, fixed problem with OptionsDescriber
commit dc2655622f71676012cf849a849e2e89fd936d26
Author: Russ Weeks <[email protected]>
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 [email protected] or file a JIRA ticket
with INFRA.
---
