[GitHub] jena pull request #511: JENA-1650: Retire jena-csv

2018-12-15 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/511 JENA-1650: Retire jena-csv Code removal, leaving only a `jena-csv/README.md`. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena retire-jena-csv

[GitHub] jena pull request #510: JENA-1649: Retire jena-fuseki1

2018-12-15 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/510 JENA-1649: Retire jena-fuseki1 Code removal, leaving only a `jena-fuseki1/README.md`. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena retire

[GitHub] jena pull request #509: JENA-1651: Specific counter for tracking active tran...

2018-12-14 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/509 JENA-1651: Specific counter for tracking active transactions. This only happens occasionally even under high update load and is harmless, only leading to a spurious log message. You can merge this pull

[GitHub] jena pull request #508: JENA-1649, JENA-1650: Remove jena-csv and jena-fusek...

2018-12-13 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/508 JENA-1649, JENA-1650: Remove jena-csv and jena-fuseki1 from the build This removed jena-csv and jena-fuseki1 from the top POM. You can merge this pull request into a Git repository by running

[GitHub] jena issue #473: Extendable versions of classes for printing result sets

2018-12-12 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/473 @hartig - see PR #507 ; that should allow you to `ResultSetWriterRegistry.register(SPARQLResultSetText, )` to set your own text output engine. For grabbing the prologue, see: https

[GitHub] jena pull request #507: JENA-1648: First class text format for result sets.

2018-12-12 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/507 JENA-1648: First class text format for result sets. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena text-format Alternatively you can

[GitHub] jena pull request #506: Cleanup of comments and javadoc. Trivial fixes.

2018-12-12 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/506 Cleanup of comments and javadoc. Trivial fixes. Remove one "abstract" in an interface. Correct QueryExecUtils handling of RDF-format result sets. You can merge this pull request into a Git

[GitHub] jena issue #473: Extendable versions of classes for printing result sets

2018-12-12 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/473 Jena is already extensible for results formats. See `ResultSetWriterRegistry` and `ResultSetReaderRegistry`. I'm sorry there is old code around but, well, time ... . Adding

[GitHub] jena issue #473: Extendable versions of classes for printing result sets

2018-12-10 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/473 I got sidetracked :-) last time. The code in this area is ancient. Text format isn't like the other formats because (1) it is isn't self-contained, It needs prefixes for abbreviation and (2

[GitHub] jena pull request #505: JENA-1647: Allow '/' and '~' in URNs

2018-12-07 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/505 JENA-1647: Allow '/' and '~' in URNs Remove copies of XML files in in src/main/java You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena urn

[GitHub] jena issue #501: JENA-1643: Use transactional dataset in clear().

2018-11-30 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/501 And review approvals? ---

[GitHub] jena pull request #501: JENA-1643: Use transactional dataset in clear().

2018-11-30 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/501#discussion_r237839306 --- Diff: jena-tdb/src/main/java/org/apache/jena/tdb/store/GraphTDB.java --- @@ -171,7 +171,10 @@ protected final int graphBaseSize() { @Override

[GitHub] jena pull request #501: JENA-1643: Use transactional dataset in clear().

2018-11-30 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/501 JENA-1643: Use transactional dataset in clear(). The fix is the one line change in method `clear()`. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] jena issue #500: JENA-1642: Check has value for header instead of is true

2018-11-29 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/500 > it appears that the maven tools are no being built as part of the main project. Is that the case? Yes, I'm afraid it is. Some tests fail when used with Apache parent >19 (I just c

[GitHub] jena pull request #499: JENA-1641: Iter.flatMap.

2018-11-27 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/499 JENA-1641: Iter.flatMap. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena iter-flatmap Alternatively you can review and apply these changes

[GitHub] jena issue #473: Extendable versions of classes for printing result sets

2018-11-26 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/473 Recorded as https://issues.apache.org/jira/browse/JENA-1640. ---

[GitHub] jena issue #473: Extendable versions of classes for printing result sets

2018-11-26 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/473 Sorry this has taken so long to get round to. ---

[GitHub] jena issue #498: JENA-1639: Model.getList and Statement.getList

2018-11-25 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/498 Ditto! ---

[GitHub] jena issue #498: JENA-1639: Model.getList and Statement.getList

2018-11-25 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/498 `interface Model extends ModelCon` (I think to put many of the convenience wrappings in a different place i.e. `ModelCon`) ---

[GitHub] jena pull request #498: JENA-1639: Model.getList and Statement.getList

2018-11-25 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/498#discussion_r236089837 --- Diff: jena-core/src/main/java/org/apache/jena/rdf/model/ModelCon.java --- @@ -111,38 +111,62 @@ * It is in effect an unsafe downcast

[GitHub] jena pull request #498: JENA-1639: Model.getList and Statement.getList

2018-11-25 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/498 JENA-1639: Model.getList and Statement.getList Attempted to use the local code style at each addition point. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] jena pull request #497: JENA-1637: Default hint language for the by-lang par...

2018-11-23 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/497 JENA-1637: Default hint language for the by-lang parse commands. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena cmd-lang Alternatively you

[GitHub] jena pull request #496: JENA-1638: Fuseki cleanup.

2018-11-23 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/496 JENA-1638: Fuseki cleanup. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena fuseki-misc Alternatively you can review and apply

[GitHub] jena pull request #495: JENA-1632: https support in SOH scripts

2018-11-22 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/495 JENA-1632: https support in SOH scripts This version of SOH should work with https endpoints. However, it does not follow redirections so http->https does not work. You can merge this p

[GitHub] jena pull request #494: Don't forget StoreParams.

2018-11-21 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/494 Don't forget StoreParams. StoreParams argument was not being passed on. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena tdb2-params

[GitHub] jena pull request #493: JENA-1633: MINUS when the LHS has an unbound but men...

2018-11-21 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/493 JENA-1633: MINUS when the LHS has an unbound but mentioned variable. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena minus Alternatively

[GitHub] jena pull request #492: JENA-1623: Endpoint access control lists

2018-11-21 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/492#discussion_r235441879 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/build/FusekiBuilder.java --- @@ -70,16 +69,42 @@ public static void

[GitHub] jena pull request #492: JENA-1623: Endpoint access control lists

2018-11-21 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/492#discussion_r235441851 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/auth/AuthPolicy.java --- @@ -0,0 +1,39 @@ +/* + * Licensed to the Apache

[GitHub] jena pull request #492: JENA-1623: Endpoint access control lists

2018-11-21 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/492 JENA-1623: Endpoint access control lists This PR adds endpoint access control list, to go with the existing support for server and dataset ACLs. With endpoint ACLs, the configuration can give

[GitHub] jena issue #489: Merged store literals once for JENA-1630

2018-11-17 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/489 This is what I see (I'm not the PR contributor presumably that's the difference): ![screenshot from 2018-11-17 18-03-33](https://user-images.githubusercontent.com/101362/48664207-295f3e80-ea93

[GitHub] jena issue #491: merge of IndexErrorOption

2018-11-17 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/491 Looking at other recent PRs, it looks like they are pulled from your BDRC repo then close the PR. This is why they are "closed", not "merged". The documented workflow is to m

[GitHub] jena issue #489: Merged store literals once for JENA-1630

2018-11-16 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/489 I don't see that message. May be it has to do with how the Apache master sync's with the GH mirror. The commit message with the close happens first, while the commits themselves haven't yet fully gone

[GitHub] jena issue #489: Merged store literals once for JENA-1630

2018-11-16 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/489 Where do you get "unmerged commits"? Just specualtion: The workflow at https://cwiki.apache.org/confluence/display/JENA/Commit+Workflow+for+Github-ASF has the PR pulled from the

[GitHub] jena issue #490: JENA-1635 Fix invalid Automatic-Module-Name values

2018-11-16 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/490 PR #473 touches on these modules. Do you mind if I merge this PR into PR #473? And "fuseki", not "fuseki2" as Fuseki1 is now well and truly legacy only? ---

[GitHub] jena pull request #488: Merged Lucene upgrade. This resolves JENA-1621

2018-11-06 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/488#discussion_r231081048 --- Diff: jena-text-es/pom.xml --- @@ -140,12 +140,13 @@ com.github.alexcojocaru elasticsearch-maven-plugin

[GitHub] jena pull request #487: JENA-1624: Update plugins; surefire workaround

2018-11-04 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/487 JENA-1624: Update plugins; surefire workaround You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena plugin-updates Alternatively you can review

[GitHub] jena pull request #486: JENA-1623 : Fuseki server and service access control

2018-11-03 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/486 JENA-1623 : Fuseki server and service access control Includes authentication support. This is packaging up the Jetty facilities to make them more accessible for typical usage in Fuseki

[GitHub] jena pull request #484: SPARQLer

2018-10-27 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/484 SPARQLer The SPARQLer files got lost in the Fuseki reoganisation for basic->main. + other clearing up. You can merge this pull request into a Git repository by running: $ git pull ht

[GitHub] jena pull request #483: JENA-1620: Protocol timeouts

2018-10-27 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/483 JENA-1620: Protocol timeouts You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena fuseki-timeout Alternatively you can review and apply

[GitHub] jena pull request #482: JENA-1619: Completely detach results by copying a bi...

2018-10-24 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/482 JENA-1619: Completely detach results by copying a binding. cd75e6c fixes the presenting problem. c89ed12 removes a related non-working feature in TDB1/TDB2. 4dc260e is reformatting. You can

[GitHub] jena pull request #479: JENA-1515 - Ensures values added to subQuery are not...

2018-10-12 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/479#discussion_r224923241 --- Diff: jena-extras/jena-querybuilder/src/main/java/org/apache/jena/arq/querybuilder/rewriters/ElementRewriter.java --- @@ -103,17 +103,13 @@ public void visit

[GitHub] jena issue #478: JENA-1612: A lightly parallel loader plan.

2018-10-09 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/478 Includes [JENA-1616](https://issues.apache.org/jira/browse/JENA-1616). Working on the JENA-1612 "light" loader, I noticed the logging output was not right for multiple file loads. ---

[GitHub] jena pull request #479: JENA-1515 - Ensures values added to subQuery are not...

2018-10-07 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/479#discussion_r223217292 --- Diff: jena-extras/jena-querybuilder/src/main/java/org/apache/jena/arq/querybuilder/rewriters/ElementRewriter.java --- @@ -103,17 +103,13 @@ public void visit

[GitHub] jena pull request #480: JENA-1602 - added mechanism to add inserts or delete...

2018-10-07 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/480#discussion_r223217083 --- Diff: jena-extras/jena-querybuilder/.gitignore --- @@ -0,0 +1 @@ +/.project_old --- End diff -- This seem to be some issue

[GitHub] jena pull request #480: JENA-1602 - added mechanism to add inserts or delete...

2018-10-07 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/480#discussion_r223217180 --- Diff: jena-maven-tools/pom.xml --- @@ -30,7 +29,7 @@ org.apache.jena jena -3.7.0-SNAPSHOT +3.9.0-SNAPSHOT --- End

[GitHub] jena pull request #480: JENA-1602 - added mechanism to add inserts or delete...

2018-10-07 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/480#discussion_r223217155 --- Diff: jena-maven-tools/pom.xml --- @@ -20,7 +20,6 @@ 4.0.0 jena-maven-tools maven-plugin - 3.7.0-SNAPSHOT --- End diff

[GitHub] jena pull request #478: JENA-1612: A lightly parallel loader plan.

2018-10-06 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/478 JENA-1612: A lightly parallel loader plan. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena load-light Alternatively you can review

[GitHub] jena pull request #477: JENA-1610: One "Metadata" class.

2018-10-06 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/477 JENA-1610: One "Metadata" class. Command line tools: Report Jena version, not subsystems. You can merge this pull request into a Git repository by running: $ git pull https://github.co

[GitHub] jena issue #474: JENA-1609 Jena master does not build under JDK10

2018-10-03 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/474 I've picked out the .gitignore changes (not for `META-INF` - that's needed) and added them. ---

[GitHub] jena issue #476: Fix JDK10 Build (JENA-1609)

2018-10-03 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/476 Looks good. A Java 10.0.2 build `mvn clean install` gets passed `jena-jdbc`: ``` [ERROR] Failed to execute goal on project jena-elephas-common: Could not resolve dependencies for project

[GitHub] jena issue #474: JENA-1609 Jena master does not build under JDK10

2018-10-03 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/474 `10 < 11`. All these issues are in Java11 because Java 9 & 10 were the big changes and Java 11 the consolidation. ---

[GitHub] jena issue #474: JENA-1609 Jena master does not build under JDK10

2018-10-03 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/474 "should be used with JDK 1.8" has several facets. There are different levels of Java11 usage: the project discussions back when it was Java9: [dev@ message](https://lists.

[GitHub] jena pull request #474: JENA-1609 Jena master does not build under JDK10

2018-10-03 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/474#discussion_r222399187 --- Diff: jena-jdbc/jena-jdbc-core/pom.xml --- @@ -39,17 +39,28 @@ 3.10.0-SNAPSHOT

[GitHub] jena pull request #474: JENA-1609 Jena master does not build under JDK10

2018-10-03 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/474#discussion_r43798 --- Diff: jena-jdbc/jena-jdbc-core/pom.xml --- @@ -39,17 +39,28 @@ 3.10.0-SNAPSHOT

[GitHub] jena issue #473: Extendable versions of classes for printing result sets

2018-10-03 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/473 On the question of mapping `ResultsFormat.FMT_TEXT`: IIRC it is because text is only an output format and not an input format (but then CSV is also only an output format). When Jena 3.9.0

[GitHub] jena pull request #474: JENA-1609 Jena master does not build under JDK10

2018-10-03 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/474#discussion_r33579 --- Diff: jena-jdbc/jena-jdbc-core/pom.xml --- @@ -39,17 +39,28 @@ 3.10.0-SNAPSHOT

[GitHub] jena issue #474: JENA-1609 Jena master does not build under JDK10

2018-10-03 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/474 @rvesse - The current plugin in the build is 3.0.0. `` didn't change the situation. Can you given an example or point to one? (the online plugin doc does not talk about `` Profiles: that would

[GitHub] jena issue #474: JENA-1609 Jena master does not build under JDK10

2018-10-03 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/474 @lewismc > @afs how about we propose the change to > > ``` > -html4 > ``` > This would produce JDK WARNING's under several environment's however would at least

[GitHub] jena issue #474: JENA-1609 Jena master does not build under JDK10

2018-10-02 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/474 I'm trying to get a single build configuration for both java10 and java8. I can't get Jena to build with java8 if `-html5` is set. It is a problem in the full build because skipping

[GitHub] jena pull request #474: JENA-1609 Jena master does not build under JDK10

2018-10-02 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/474#discussion_r221928887 --- Diff: jena-jdbc/jena-jdbc-core/pom.xml --- @@ -39,17 +39,28 @@ 3.10.0-SNAPSHOT

[GitHub] jena pull request #474: JENA-1609 Jena master does not build under JDK10

2018-10-01 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/474#discussion_r221744935 --- Diff: jena-jdbc/jena-jdbc-core/pom.xml --- @@ -39,17 +39,28 @@ 3.10.0-SNAPSHOT

[GitHub] jena issue #474: JENA-1609 Jena master does not build under JDK10

2018-10-01 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/474 See https://builds.apache.org/view/H-L/view/Jena/job/Jena_JDK11/ which builds with -Pdev and has been running for a while. It does not build jena-jdbc or jena-elephas both of which are need looking at. ---

[GitHub] jena pull request #474: JENA-1609 Jena master does not build under JDK10

2018-10-01 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/474#discussion_r221738972 --- Diff: jena-arq/pom.xml --- @@ -216,6 +216,7 @@ Licenced under the Apache License, Version 2.0

[GitHub] jena issue #470: Rename README to README.md

2018-09-28 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/470 @nirajpatel - done - thanks! ---

[GitHub] jena pull request #472: JENA-1607: Protect stdout, stderr from being closed

2018-09-25 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/472 JENA-1607: Protect stdout, stderr from being closed You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena logging-fix Alternatively you can

[GitHub] jena pull request #468: JENA-1597 : Reorganisation of Fuseki modules.

2018-09-14 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/468 JENA-1597 : Reorganisation of Fuseki modules. This PR reorgansies the Fuseki modules to separate out the core engine, regardless of use in a webapp or embedded, to make what was the "embedded&quo

[GitHub] jena pull request #466: JENA-1596: Pass original string queryResultSet, cons...

2018-08-31 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/466 JENA-1596: Pass original string queryResultSet, construct/describe/ask. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena rdfconnection-remote

[GitHub] jena issue #465: JENA-1594: Per-graph access control for Fuseki

2018-08-28 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/465 This is now code-finished. It is still a library that needs code around it to use. To make it usable without code, it will need some packaging and setup with Jetty and documentation about

[GitHub] jena issue #465: JENA-1594: Per-graph access control for Fuseki

2018-08-28 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/465 Thanks - fixed applied. ---

[GitHub] jena issue #465: JENA-1594: Per-graph access control for Fuseki

2018-08-24 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/465 This PR is not yet completely finished. It is put out now to give it early visibility. I have some development checking to do, some code tidying and documentation but the tests are reasonably

[GitHub] jena pull request #465: JENA-1594: Per-graph access control for Fuseki

2018-08-24 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/465 JENA-1594: Per-graph access control for Fuseki This adds a new module `jena-fuseki-access` that contains code for filtering data in SPARQL Query, SPARQL Graph Store Protocol, and plain quads access

[GitHub] jena pull request #463: Cleanup

2018-08-20 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r211266636 --- Diff: jena-arq/src/main/java/org/apache/jena/sparql/core/DynamicDatasets.java --- @@ -56,8 +55,8 @@ public static DatasetGraph dynamicDataset

[GitHub] jena pull request #464: NodeUtils convert: set and list versions

2018-08-20 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/464 NodeUtils convert: set and list versions In response to PR #463 where a [comment](https://github.com/apache/jena/pull/463#discussion_r211191248) mentions set/collections issues. You can merge this pull

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210921706 --- Diff: jena-fuseki2/jena-fuseki-embedded/src/main/java/org/apache/jena/fuseki/embedded/FusekiServer.java --- @@ -455,30 +527,36 @@ private static void

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210891477 --- Diff: jena-fuseki2/jena-fuseki-embedded/src/main/java/org/apache/jena/fuseki/embedded/FusekiServer.java --- @@ -455,30 +527,36 @@ private static void

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/463#discussion_r210889580 --- Diff: jena-base/src/main/java/org/apache/jena/atlas/lib/ListUtils.java --- @@ -34,49 +34,40 @@ { private ListUtils() {} -public

[GitHub] jena pull request #463: Cleanup

2018-08-17 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/463 Cleanup This is a collection of small items, accumulated while working with the source code attached to a project. * Various comments/javadoc added and fixed and other code cleanup. * make

[GitHub] jena pull request #459: Fuseki code maintenance.

2018-08-13 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/459#discussion_r209542012 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/mgt/ActionDatasets.java --- @@ -288,28 +294,77 @@ protected void execDeleteItem

[GitHub] jena pull request #459: Fuseki code maintenance.

2018-08-11 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/459#discussion_r209422540 --- Diff: jena-base/src/main/java/org/apache/jena/atlas/io/IO.java --- @@ -364,4 +369,31 @@ public static String uniqueFilename(String directory, String base

[GitHub] jena pull request #459: Fuseki code maintenance.

2018-08-11 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/459#discussion_r209422526 --- Diff: jena-db/jena-tdb2/src/main/java/org/apache/jena/tdb2/sys/TDBInternal.java --- @@ -176,6 +176,23 @@ public static synchronized void expel(DatasetGraph

[GitHub] jena pull request #459: Fuseki code maintenance.

2018-08-11 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/459#discussion_r209421919 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/server/DataService.java --- @@ -151,50 +151,24 @@ public long getRequestsBad

[GitHub] jena pull request #459: Fuseki code maintenance.

2018-08-11 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/459#discussion_r209421906 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/servlets/HttpAction.java --- @@ -18,8 +18,8 @@ package

[GitHub] jena pull request #459: Fuseki code maintenance.

2018-08-11 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/459#discussion_r209421902 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_Upload.java --- @@ -20,35 +20,39 @@ import static

[GitHub] jena pull request #459: Fuseki code maintenance.

2018-08-11 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/459#discussion_r209421904 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/servlets/HttpAction.java --- @@ -270,30 +269,50 @@ public boolean isTransactional

[GitHub] jena pull request #459: Fuseki code maintenance.

2018-08-11 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/459#discussion_r209421898 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_Upload.java --- @@ -20,35 +20,39 @@ import static

[GitHub] jena pull request #459: Fuseki code maintenance.

2018-08-10 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/459 Fuseki code maintenance. This follows on from PR #458 so it shows the commits for that PR in the comparison to Jena master. It is the 3 individual commits after "Delete databases and expel

[GitHub] jena pull request #458: JENA-1481, JENA-1586: Delete databases and expel fro...

2018-08-10 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/458#discussion_r209313977 --- Diff: jena-base/src/main/java/org/apache/jena/atlas/io/IO.java --- @@ -364,4 +369,31 @@ public static String uniqueFilename(String directory, String base

[GitHub] jena pull request #458: JENA-1481, JENA-1586: Delete databases and expel fro...

2018-08-10 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/458#discussion_r209313461 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/mgt/ActionDatasets.java --- @@ -288,28 +294,89 @@ protected void execDeleteItem

[GitHub] jena pull request #458: JENA-1481, JENA-1586: Delete databases and expel fro...

2018-08-10 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/458 JENA-1481, JENA-1586: Delete databases and expel from JVM You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena fuseki-db-delete Alternatively

[GitHub] jena issue #456: JENA-1585: Fuseki core reorg

2018-08-05 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/456 No problem on timing - I found I'd tripped myself up moving java resources around and tested against a setup that had old and new location materials. TravisCI caught this for me. ---

[GitHub] jena pull request #456: JENA-1585: Fuseki core reorg

2018-08-05 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/456#discussion_r207735470 --- Diff: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/build/FusekiBuildLib.java --- @@ -0,0 +1,125 @@ +/* + * Licensed

[GitHub] jena pull request #456: JENA-1585: Fuseki core reorg

2018-08-05 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/456 JENA-1585: Fuseki core reorg Class reorganisation within the the jena-fuseki-core module. jena-fuseki-embedded and jena-fuseki-basic changes show the cleaner split - no mention of the webapp

[GitHub] jena pull request #455: JENA-1584: A Javacc Turtle parser for reference.

2018-08-05 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/455 JENA-1584: A Javacc Turtle parser for reference. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena ttl-javacc Alternatively you can review

[GitHub] jena pull request #454: JENA-1583: Rule system: Util.compare

2018-08-03 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/454 JENA-1583: Rule system: Util.compare You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena rule-util Alternatively you can review and apply

[GitHub] jena issue #449: JENA-1578

2018-08-01 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/449 There is no need to use the un-parenthesed form - it is a syntactic short form and the full form can always be used. This may make the code generation easier and have less at risk of misalignment

[GitHub] jena pull request #453: JENA-1581: TDB1 Node Buffer Mgt.

2018-08-01 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/453 JENA-1581: TDB1 Node Buffer Mgt. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena tdb-buffer Alternatively you can review and apply

[GitHub] jena pull request #452: JENA-1580: Better error message when port in use

2018-07-29 Thread afs
GitHub user afs opened a pull request: https://github.com/apache/jena/pull/452 JENA-1580: Better error message when port in use You can merge this pull request into a Git repository by running: $ git pull https://github.com/afs/jena fuseki-port Alternatively you can review

[GitHub] jena pull request #449: JENA-1578

2018-07-28 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/449#discussion_r205953487 --- Diff: jena-arq/src/main/java/org/apache/jena/query/ParameterizedSparqlString.java --- @@ -1734,4 +1739,237 @@ public String toString

[GitHub] jena pull request #449: JENA-1578

2018-07-28 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/449#discussion_r205953412 --- Diff: jena-arq/src/main/java/org/apache/jena/query/ParameterizedSparqlString.java --- @@ -1734,4 +1739,237 @@ public String toString

[GitHub] jena issue #449: JENA-1578

2018-07-28 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/449 @ajs6f Will take a look. A particular thing to be careful of with ParameterizedSparqlString in general is injection attacks. cc @rvesse ---

  1   2   3   4   5   6   7   8   9   10   >