[GitHub] [jena] afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-28 Thread GitBox
afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-496509838 I don't see any evaluation tests, only syntax tests. Data can be provided by `VALUES ?x { 1 2 3 4 5 }` and `VALUES ?x { 1 2 3 4 5 6 }`.

[GitHub] [jena] afs commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-28 Thread GitBox
afs commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r288091834 ## File path: jena-arq/testing/ARQ/Syntax/syn-arq.sh ## @@ -328,3 +328,17 @@ WHERE GRAPH ?g { ?s ?p ?o.

[GitHub] [jena] ajs6f commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-28 Thread GitBox
ajs6f commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r288087992 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java ## @@ -0,0 +1,118 @@

[GitHub] [jena] neumarcx commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-28 Thread GitBox
neumarcx commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r288081119 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java ## @@ -0,0 +1,118

[GitHub] [jena] ajs6f commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-28 Thread GitBox
ajs6f commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r288079380 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java ## @@ -0,0 +1,118 @@

[GitHub] [jena] ajs6f commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-28 Thread GitBox
ajs6f commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r288077331 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java ## @@ -0,0 +1,118 @@

[GitHub] [jena] afs merged pull request #570: JENA-1713: Switch off warning for NON_INITIAL_DOT_SEGMENT

2019-05-27 Thread GitBox
afs merged pull request #570: JENA-1713: Switch off warning for NON_INITIAL_DOT_SEGMENT URL: https://github.com/apache/jena/pull/570 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [jena] afs opened a new pull request #570: JENA-1713: Switch off warning for NON_INITIAL_DOT_SEGMENT

2019-05-25 Thread GitBox
afs opened a new pull request #570: JENA-1713: Switch off warning for NON_INITIAL_DOT_SEGMENT URL: https://github.com/apache/jena/pull/570 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [jena] afs merged pull request #569: POM maintenance

2019-05-23 Thread GitBox
afs merged pull request #569: POM maintenance URL: https://github.com/apache/jena/pull/569 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [jena] afs opened a new pull request #569: Pom maintenance

2019-05-23 Thread GitBox
afs opened a new pull request #569: Pom maintenance URL: https://github.com/apache/jena/pull/569 Covers JENA-1711 (jackson update) and JENA -1712 (use https for links). This is an automated message from the Apache Git

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-05-21 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-494426066 Merged to master! This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [jena] asfgit merged pull request #557: JENA-664 GeoSPARQL Jena

2019-05-21 Thread GitBox
asfgit merged pull request #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [jena] neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-21 Thread GitBox
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-494415007 @afs I think got this to work. did a git remote add upstream, git fetch upstream, git merge upstream/master, git commit and followed by a git push

[GitHub] [jena] afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-21 Thread GitBox
afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-494408445 `git pull upstream master` (fetch only gets the changes and does not apply them).

[GitHub] [jena] neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-21 Thread GitBox
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-494367891 @afs trying to sync the fork with git fetch upstream first. but don't see your changes in the apache/jena repo yet being transferred to the

[GitHub] [jena] afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-21 Thread GitBox
afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-494359631 @neumarcx - I don't see any evaluation tests. Some tests with `VALUES ?x { 1 2 3 4 5 }` and `VALUES ?x { 1 2 3 4 5 6 }` (the odd and even cases)

[GitHub] [jena] neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-20 Thread GitBox
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-494123808 thank you Andy. while you are in the Syntax testing section. I have noticed that syn.sh was not executable on checkout. might be a good idea to

[GitHub] [jena] afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-20 Thread GitBox
afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-494097841 I've updated the syntax tests so that they are all script written and also fixed bit rot where the tests were right but the script to create them was

[GitHub] [jena] neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-20 Thread GitBox
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-494062956 OK yes, looks like these tests haven't been touched for quite some time on syn.sh throwing errors for the bindings tests as mentioned in the

[GitHub] [jena] afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-20 Thread GitBox
afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-493950749 `syn-func.sh` and loaded with `source syn-func.sh` in `syn.sh`. This is an

[GitHub] [jena] neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-18 Thread GitBox
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-493658989 where do the fname and testGood commands come from? This is an automated message

[GitHub] [jena] afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-17 Thread GitBox
afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-493518708 The driver script is `syn.sh` - it sets the output directory by cd'ing into it and running the script: ``` # ARQ ( cd Syntax-ARQ

[GitHub] [jena] neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-17 Thread GitBox
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-493505146 > Syntax test: the syntax manifest and target query files are generated from various "syn-*.sh" files. (But that is a bash shell script to be

[GitHub] [jena] neumarcx commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-17 Thread GitBox
neumarcx commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r285185337 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java ## @@ -0,0 +1,123

[GitHub] [jena] neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-17 Thread GitBox
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-493487384 > One observation is that the Commons Math library used notes that for percentile based stats to be evaluated correctly the data should be at

[GitHub] [jena] neumarcx commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-17 Thread GitBox
neumarcx commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r285162989 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedianDistinct.java ## @@ -0,0

[GitHub] [jena] afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-17 Thread GitBox
afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-493483043 Syntax test: the syntax manifest and target query files are generated from various "syn-*.sh" files. (But that is a bash shell script to be run.)

[GitHub] [jena] afs commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-17 Thread GitBox
afs commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r285159993 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedianDistinct.java ## @@ -0,0

[GitHub] [jena] afs commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-17 Thread GitBox
afs commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r285159912 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java ## @@ -0,0 +1,125 @@ +/*

[GitHub] [jena] afs commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-17 Thread GitBox
afs commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r285153474 ## File path: jena-arq/pom.xml ## @@ -113,6 +113,12 @@ commons-lang3 + + org.apache.commons

[GitHub] [jena] ajs6f commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-17 Thread GitBox
ajs6f commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r285133389 ## File path: jena-arq/Grammar/sparql_11.jj ## @@ -1318,6 +1318,8 @@ Expr Aggregate() : { Aggregator agg = null ;

[GitHub] [jena] neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-16 Thread GitBox
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-493263872 Where is the best place for the ARQ median function test in the testing folder jena/jena-arq/testing/ARQ/Syntax?

[GitHub] [jena] neumarcx commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-16 Thread GitBox
neumarcx commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r284931581 ## File path: jena-arq/Grammar/sparql_11.jj ## @@ -1318,6 +1318,8 @@ Expr Aggregate() : { Aggregator agg =

[GitHub] [jena] rvesse commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-16 Thread GitBox
rvesse commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-493031741 One observation is that the Commons Math library used notes that for percentile based stats to be evaluated correctly the data should be at least

[GitHub] [jena] rvesse commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-16 Thread GitBox
rvesse commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r284658058 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java ## @@ -0,0 +1,123 @@

[GitHub] [jena] rvesse commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-16 Thread GitBox
rvesse commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r284659933 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java ## @@ -0,0 +1,123 @@

[GitHub] [jena] rvesse commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-16 Thread GitBox
rvesse commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r284656594 ## File path: jena-arq/Grammar/sparql_11.jj ## @@ -1318,6 +1318,8 @@ Expr Aggregate() : { Aggregator agg = null

[GitHub] [jena] rvesse commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-16 Thread GitBox
rvesse commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r284656165 ## File path: jena-arq/Grammar/master.jj ## @@ -2045,6 +2045,9 @@ Expr Aggregate() : { Aggregator agg = null ;

[GitHub] [jena] rvesse commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-16 Thread GitBox
rvesse commented on a change in pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#discussion_r284658880 ## File path: jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java ## @@ -0,0 +1,123 @@

[GitHub] [jena] neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-15 Thread GitBox
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-492696407 can you ignore the test case and the last PR update and succesfully build? This

[GitHub] [jena] neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-15 Thread GitBox
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-492690620 OK, I don't but the error looks good. median should not pass on SPARQL1.1. just updated PR.

[GitHub] [jena] ajs6f commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-15 Thread GitBox
ajs6f commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-492664185 @neumarcx I'm getting a test error trying to build this. Are you seeing what I show below? ``` ... [INFO] Running

[GitHub] [jena] neumarcx opened a new pull request #568: Add Aggregate Median to SPARQL ARQ syntax

2019-05-15 Thread GitBox
neumarcx opened a new pull request #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568 this adds aggregate MEDIAN as keyword to the SPARQL ARQ syntax This is an automated message from

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-05-08 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-490459291 I've merged the PR onto to working branch [geosparql](https://github.com/apache/jena/tree/geosparql) with most of the post-merge changes applied. Outstanding:

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-05-07 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-490167509 @galbiston - Thanks. The license text was passing the RAT checks but there was extra wording like "Copyright the authors". Not wrong, but a variation. Copyright and IP

[GitHub] [jena] galbiston commented on issue #557: JENA-664 GeoSPARQL Jena

2019-05-06 Thread GitBox
galbiston commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-489695240 Apologies for the license issue. I thought I'd tidied them all up. I've also removed the lines of test name output, removed the commented out test results and

[GitHub] [jena] afs merged pull request #567: JENA-1709: Password file implies logged-in users

2019-05-02 Thread GitBox
afs merged pull request #567: JENA-1709: Password file implies logged-in users URL: https://github.com/apache/jena/pull/567 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [jena] rvesse commented on a change in pull request #567: JENA-1709: Password file implies logged-in users

2019-05-02 Thread GitBox
rvesse commented on a change in pull request #567: JENA-1709: Password file implies logged-in users URL: https://github.com/apache/jena/pull/567#discussion_r280333708 ## File path: jena-fuseki2/jena-fuseki-main/src/main/java/org/apache/jena/fuseki/main/FusekiServer.java

[GitHub] [jena] afs opened a new pull request #567: JENA-1709: Password file implies logged-in users

2019-05-01 Thread GitBox
afs opened a new pull request #567: JENA-1709: Password file implies logged-in users URL: https://github.com/apache/jena/pull/567 and some other minor fixes and log warnings. This is an automated message from the Apache Git

[GitHub] [jena] afs commented on issue #546: JENA-1687: refine NodeFactory return types

2019-04-30 Thread GitBox
afs commented on issue #546: JENA-1687: refine NodeFactory return types URL: https://github.com/apache/jena/pull/546#issuecomment-487892216 Thanks! This is an automated message from the Apache Git Service. To respond to the

[GitHub] [jena] jmkeil commented on issue #546: JENA-1687: refine NodeFactory return types

2019-04-30 Thread GitBox
jmkeil commented on issue #546: JENA-1687: refine NodeFactory return types URL: https://github.com/apache/jena/pull/546#issuecomment-487882901 Yes. The change is unnecessary. This is an automated message from the Apache Git

[GitHub] [jena] jmkeil closed pull request #546: JENA-1687: refine NodeFactory return types

2019-04-30 Thread GitBox
jmkeil closed pull request #546: JENA-1687: refine NodeFactory return types URL: https://github.com/apache/jena/pull/546 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [jena] afs merged pull request #561: JENA-1572: Remove L in modules that don't need them

2019-04-30 Thread GitBox
afs merged pull request #561: JENA-1572: Remove L in modules that don't need them URL: https://github.com/apache/jena/pull/561 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [jena] afs commented on issue #546: JENA-1687: refine NodeFactory return types

2019-04-30 Thread GitBox
afs commented on issue #546: JENA-1687: refine NodeFactory return types URL: https://github.com/apache/jena/pull/546#issuecomment-487873750 Given the other PRs and discussion on [JENA-1687](https://issues.apache.org/jira/browse/JENA-1687), I don't think we should make the change from the

[GitHub] [jena] afs edited a comment on issue #547: JENA-1688: add methods Node#asBlank, Node#asConcret, Node#asLiteral, Node#asURI, Node#asVariable

2019-04-30 Thread GitBox
afs edited a comment on issue #547: JENA-1688: add methods Node#asBlank, Node#asConcret, Node#asLiteral, Node#asURI, Node#asVariable URL: https://github.com/apache/jena/pull/547#issuecomment-476405301 This seem related to PR#546 and the discussion on

[GitHub] [jena] afs merged pull request #566: Update maven-bundle-plugin

2019-04-30 Thread GitBox
afs merged pull request #566: Update maven-bundle-plugin URL: https://github.com/apache/jena/pull/566 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-28 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487384847 @galbiston -- Sorry - I've missed something. Some of the files have the Apache License for contributed software, (example `GeoSPARQLConfig.java`) and some have

[GitHub] [jena] desruisseaux commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-27 Thread GitBox
desruisseaux commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487301304 About precision, note that `Math.sin`, `cos` _etc._ are not fully deterministic even in Java. The specification said only that the result must be within 1 or 2

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-27 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487297031 ## Status (2019-04-27) ### Next I've made some changes locally and got the build to work with tests. All the changes are detailed below. If the

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487110358 "mvn clean test" takes 12s. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [jena] ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487099217 Cool, then we can build it once and pass the location around via a Maven property. This is an

[GitHub] [jena] desruisseaux commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
desruisseaux commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487098680 The performance interrogation was because SIS may try to create the EPSG geodetic database in that directory (depending if `org.apache.sis.non-free:sis-epsg`

[GitHub] [jena] ajs6f edited a comment on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
ajs6f edited a comment on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487095451 @desruisseaux We have I/O tests that take five minutes to run-- I'm not sure that making a few empty directories is going to change our build time very much,

[GitHub] [jena] ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487095451 @desruisseaux We have I/O tests that take five minutes to run-- I'm not sure that making a few empty directories is going to change our build time very much, but if

[GitHub] [jena] desruisseaux commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
desruisseaux commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487092706 `SIS_DATA` is an environment variable, while Maven Surefire `` is (in my understanding) for Java properties. Another issue is that pointing to

[GitHub] [jena] ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487088627 @afs Can we not use some Maven for that? We could put an empty `SIS_DATA` directory in `src/test/resources` and then: ```

[GitHub] [jena] afs edited a comment on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs edited a comment on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487086401 @ajs6f - create an empty test directory, point `SIS_DATA` to it. However, it takes some fixing to get the module running at the moment.

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487086401 @ajs6f - create an empty test directory, point `SIS_DATA` to it. However, it takes some fixing to get the module at the moment.

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487081192 @desruisseaux - perfect! Thank you. This is an automated message from the Apache Git Service. To respond

[GitHub] [jena] ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487079821 @afs No, sorry, I meant to ask you to point me at what we want to run _before_ the tests: the setup step.

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487079179 > @afs Can you point me at what we actually want to run? The tests are in src/test as usual and run with surefire finding \@Test. It is not urgent at this

[GitHub] [jena] desruisseaux commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
desruisseaux commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487078186 Actually, while preparing the email to OGC, I found answer to the question in their [legal FAQ](http://www.opengeospatial.org/ogc/legalfaq#DTD). Quoting them:

[GitHub] [jena] ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487070553 @afs Can you point me at what we actually want to run? Are we talking about setting up some of these GIS reference databases like the stuff in `SIS_DATA`, or

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487070583 @desruisseaux, if you could ask OGC that would be great. It makes sense to restrict documents that are the definitive specifications (as do most standards organisations

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487069142 @ajs6f It is currently working by surefire finding the tests so there is no place to add a suite-wide step. If we add such as step with `RunWith` / `SuiteClasses` we

[GitHub] [jena] desruisseaux commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
desruisseaux commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487064619 Hello @afs . The question about the exact license governing the RDF file is a good one. I confirm that OGC "Software" license is BSD-like and classified as

[GitHub] [jena] ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487061029 @afs This PR is so big I cannot even load the list of files in Github's browser. Since we are still using JUnit 4 we can write a

[GitHub] [jena] afs edited a comment on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs edited a comment on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487046910 Next issue: `geosparql_vocab_all_v1_0_1_updated.rdf` It references the OGC license for documents http://www.opengeospatial.org/legal/ But is it

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487046910 Next issue: `geosparql_vocab_all_v1_0_1_updated.rdf` It references the OGC license for documents http://www.opengeospatial.org/legal/ But is it "document" or

[GitHub] [jena] afs edited a comment on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs edited a comment on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487025047 > Set the SIS_DATA environment variable to the path of an initially empty directory (preferred choice). Ignoring that, and running with a java8 JDK

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-26 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-487025047 I'm using this in the POM for the tests: ``` org.apache.sis.non-free sis-embedded-data 0.8 test

[GitHub] [jena] afs opened a new pull request #566: Update maven-bundle-plugin

2019-04-26 Thread GitBox
afs opened a new pull request #566: Update maven-bundle-plugin URL: https://github.com/apache/jena/pull/566 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [jena] afs merged pull request #564: JENA-1708: Remove unused javadoc artifact depedencies

2019-04-26 Thread GitBox
afs merged pull request #564: JENA-1708: Remove unused javadoc artifact depedencies URL: https://github.com/apache/jena/pull/564 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [jena] afs merged pull request #565: Update javadoc configuration to support jdk11 builds

2019-04-26 Thread GitBox
afs merged pull request #565: Update javadoc configuration to support jdk11 builds URL: https://github.com/apache/jena/pull/565 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [jena] acoburn opened a new pull request #565: Update javadoc configuration to support jdk11 builds

2019-04-25 Thread GitBox
acoburn opened a new pull request #565: Update javadoc configuration to support jdk11 builds URL: https://github.com/apache/jena/pull/565 This updates the javadoc configuration so that JDK11-based builds will succeed at the javadoc stage. As with the current maven configuration, the Java

[GitHub] [jena] ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-25 Thread GitBox
ajs6f commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-486723449 Hello, @galbiston-- can you explain a bit what you mean by "I don't think there is an equivalent for Maven for test only." Are we trying to put resources on the

[GitHub] [jena] afs opened a new pull request #564: JENA-1708: Remove unused javadoc artifact depedencies

2019-04-25 Thread GitBox
afs opened a new pull request #564: JENA-1708: Remove unused javadoc artifact depedencies URL: https://github.com/apache/jena/pull/564 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [jena] afs merged pull request #560: JENA-1706: Handle "QueryParser"; suppress logging WARN

2019-04-25 Thread GitBox
afs merged pull request #560: JENA-1706: Handle "QueryParser"; suppress logging WARN URL: https://github.com/apache/jena/pull/560 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [jena] afs commented on issue #560: JENA-1706: Handle "QueryParser"; suppress logging WARN

2019-04-25 Thread GitBox
afs commented on issue #560: JENA-1706: Handle "QueryParser"; suppress logging WARN URL: https://github.com/apache/jena/pull/560#issuecomment-486602867 This follows on from #536. This is an automated message from the

[GitHub] [jena] afs merged pull request #563: JENA-1707: Skip building modules unless java8

2019-04-25 Thread GitBox
afs merged pull request #563: JENA-1707: Skip building modules unless java8 URL: https://github.com/apache/jena/pull/563 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [jena] afs commented on issue #563: JENA-1707: Skip building modules unless java8

2019-04-25 Thread GitBox
afs commented on issue #563: JENA-1707: Skip building modules unless java8 URL: https://github.com/apache/jena/pull/563#issuecomment-486601549 HADOOP-15338 This is an automated message from the Apache Git Service. To

[GitHub] [jena] galbiston commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-24 Thread GitBox
galbiston commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-486419457 I had the SIS_DATA dependency resolved in Gradle for testing both modules by "testImplementation "org.apache.sis.non-free:sis-embedded-data:$sisVersion". I don't

[GitHub] [jena] afs opened a new pull request #563: JENA-1707: Skip building modules unless java8

2019-04-24 Thread GitBox
afs opened a new pull request #563: JENA-1707: Skip building modules unless java8 URL: https://github.com/apache/jena/pull/563 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [jena] afs opened a new pull request #562: JENA-1695: Modular storage interface and framework

2019-04-24 Thread GitBox
afs opened a new pull request #562: JENA-1695: Modular storage interface and framework URL: https://github.com/apache/jena/pull/562 JENA-1695: Modular storage interface and framework This is the first of two pull requests. This is the framework and adjusting the code base

[GitHub] [jena] afs opened a new pull request #561: JENA-1572: Remove L in modules that don't need them

2019-04-24 Thread GitBox
afs opened a new pull request #561: JENA-1572: Remove L in modules that don't need them URL: https://github.com/apache/jena/pull/561 Work in JENA-1572. This removes LICENSE and NOTICE files that arne't needed for a modules source code.

[GitHub] [jena] afs opened a new pull request #560: JENA-1706: Handle "QueryParser"; suppress logging WARN

2019-04-24 Thread GitBox
afs opened a new pull request #560: JENA-1706: Handle "QueryParser"; suppress logging WARN URL: https://github.com/apache/jena/pull/560 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [jena] afs merged pull request #558: Support configuring syntax for Parameterized SPARQL (JENA-1705)

2019-04-24 Thread GitBox
afs merged pull request #558: Support configuring syntax for Parameterized SPARQL (JENA-1705) URL: https://github.com/apache/jena/pull/558 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [jena] afs merged pull request #559: JENA-1659: Improve tests for western hemisphere timezones

2019-04-24 Thread GitBox
afs merged pull request #559: JENA-1659: Improve tests for western hemisphere timezones URL: https://github.com/apache/jena/pull/559 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [jena] acoburn opened a new pull request #559: JENA-1659: Improve tests for western hemisphere timezones

2019-04-23 Thread GitBox
acoburn opened a new pull request #559: JENA-1659: Improve tests for western hemisphere timezones URL: https://github.com/apache/jena/pull/559 This adjusts the timezone function to pass when run in a timezone to the west of GMT. This is a follow-in to JENA-1659

[GitHub] [jena] afs commented on issue #557: JENA-664 GeoSPARQL Jena

2019-04-23 Thread GitBox
afs commented on issue #557: JENA-664 GeoSPARQL Jena URL: https://github.com/apache/jena/pull/557#issuecomment-485901035 I have found http://sis.apache.org/epsg.html and got the tests running locally. Sorry for the noise above. Test pass! Some noise to be tuned out from

<    5   6   7   8   9   10   11   12   >