New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RYA-284: updated to support bloom filter #169
Conversation
Can one of the admins verify this patch? |
Can you submit another PR with only the required changes? There are a lot of whitespace and final changes. Normally I would power through this, but the github "ignore whitespace" hack doesn't seem to be working. |
protected void fillScanner(ScannerBase scanner, RyaURI context, String qualifier, Long ttl, Long currentTime, | ||
TripleRowRegex tripleRowRegex, RdfCloudTripleStoreConfiguration conf) throws IOException { | ||
if (context != null && qualifier != null) { | ||
private static Text createText(final byte[] data, final RyaType ryaType) throws RyaTypeResolverException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"appendType" would be a better name for this method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay. Will put up a new PR with the new method name.
Range range = new Range(new Text(byteRange.getStart()), new Text(byteRange.getEnd())); | ||
if ((stmt.getSubject() != null) && (stmt.getPredicate() != null) && (stmt.getObject() != null)) { | ||
range = Range.exact(createText(byteRange.getStart(), stmt.getObject())); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does byteRange.getStart()
ever have the type appended to it at this point? (if so, then this won't work)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's what createText (current method name) is supposed to do.
Closed and will submit a new PR as requested. |
Description
Updated dao/accumulo.rya to support the use of bloom filter
Tests
no new tests
Links
JIRA -> RYA-284
People To Review
@amihalik