[
https://issues.apache.org/jira/browse/JENA-512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13742523#comment-13742523
]
Hudson commented on JENA-512:
-----------------------------
SUCCESS: Integrated in Jena_Development_Test #845 (See
[https://builds.apache.org/job/Jena_Development_Test/845/])
JENA-512
Put algebra function names in Tags.
Add equivalent builders for expessions by function name.
Use constants in E_* classes.
Rename printName as functionName.
Some tests. (andy: rev 1514832)
* /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_Add.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_Conditional.java
* /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_Divide.java
* /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_Equals.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_GreaterThan.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_GreaterThanOrEqual.java
* /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_LessThan.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_LessThanOrEqual.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_LogicalAnd.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_LogicalNot.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_LogicalOr.java
* /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_Multiply.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_NotEquals.java
* /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_NotOneOf.java
* /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_OneOf.java
* /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_SameTerm.java
* /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_Subtract.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_UnaryMinus.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/E_UnaryPlus.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/expr/ExprFunction.java
* /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/sse/Tags.java
*
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/sse/builders/BuilderExpr.java
*
/jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/syntax/TestSSE_Builder.java
> SSE Tags are used inconsistently
> --------------------------------
>
> Key: JENA-512
> URL: https://issues.apache.org/jira/browse/JENA-512
> Project: Apache Jena
> Issue Type: Bug
> Components: ARQ
> Affects Versions: Jena 2.10.1
> Reporter: Rob Vesse
> Assignee: Rob Vesse
> Priority: Minor
> Labels: algebra, sse
> Fix For: Jena 2.10.2
>
>
> In some of our more recent code we are trying to map functions into a
> messaging data structure by inspecting the function symbol. However we have
> found that there are some inconsistencies in the symbols.
> Namely that the capitalization and punctuation are not consistent, likely
> this cannot change as it would mean breaking many existing algebra
> examples/tests and systems like ours that use algebra strings internally.
> The more concerning inconsistencies are that some functions report symbols
> without using the tag constants and so some queries written report symbols
> that don't match their tag constants.
> BuilderExpr appears to get around this by doing case insensitive key lookup
> which seems very hacky
> There is also at least one function (isNumeric) which has no Tag constant and
> no SSE builder defined for it so queries containing this cannot be decoded
> from algebra.
> I plan to do two things:
> - Make expression classes return their Tags constant where they don't already
> and particularly in the cases where the two values aren't exact matches
> - Fix the isNumeric case (and any others I discover) where there is no
> registered builder for the symbol
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
