[
https://issues.apache.org/jira/browse/RYA-405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16444615#comment-16444615
]
ASF GitHub Bot commented on RYA-405:
------------------------------------
Github user ejwhite922 commented on a diff in the pull request:
https://github.com/apache/incubator-rya/pull/291#discussion_r182849095
--- Diff:
extras/indexing/src/main/java/org/apache/rya/indexing/IndexPlanValidator/ThreshholdPlanSelector.java
---
@@ -111,9 +110,9 @@ public double getCost(TupleExpr te, double indexWeight,
double commonVarWeight,
double dirProductScale;
if(queryNodeCount > nodeCount) {
- dirProductScale = 1/((double)(queryNodeCount - nodeCount));
+ dirProductScale = 1/ (queryNodeCount - nodeCount);
--- End diff --
nodeCount is a double so java forces the divisor here to be a double as
well so there's no precision lost. Casting it to a double would only be needed
if queryNodeCount and nodeCount were both int's (or other less precise number
types).
> Migrate from Sesame to rdf4j libs
> ----------------------------------
>
> Key: RYA-405
> URL: https://issues.apache.org/jira/browse/RYA-405
> Project: Rya
> Issue Type: Improvement
> Affects Versions: 3.2.12
> Reporter: Jorge Machado
> Assignee: Eric White
> Priority: Major
> Fix For: 4.0.0
>
>
> I have migrated all the project to the newer libs from rdf4j instead of using
> the old ones.
> Can someone take a look ?
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
