[GitHub] jena issue #203: JsonLDReader: possibility to override the @context

2017-01-08 Thread fpservant
Github user fpservant commented on the issue: https://github.com/apache/jena/pull/203 @ajs6f thanks for the review. I made most (all ?) of the changes that you suggested. Remains the problem "loading remote context failed: http://schema.org/; - unrelated to this PR per se.

[GitHub] jena pull request #203: JsonLDReader: possibility to override the @context

2017-01-08 Thread fpservant
Github user fpservant commented on a diff in the pull request: https://github.com/apache/jena/pull/203#discussion_r95095721 --- Diff: jena-arq/src/test/java/org/apache/jena/riot/TestJsonLDReader.java --- @@ -0,0 +1,81 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] jena pull request #203: JsonLDReader: possibility to override the @context

2017-01-08 Thread fpservant
Github user fpservant commented on a diff in the pull request: https://github.com/apache/jena/pull/203#discussion_r95095597 --- Diff: jena-arq/src/test/java/org/apache/jena/riot/TestJsonLDReader.java --- @@ -0,0 +1,81 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] jena pull request #203: JsonLDReader: possibility to override the @context

2017-01-08 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/203#discussion_r95095064 --- Diff: jena-arq/src/test/java/org/apache/jena/riot/TestJsonLDReader.java --- @@ -0,0 +1,81 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] jena pull request #203: JsonLDReader: possibility to override the @context

2017-01-08 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/203#discussion_r95095114 --- Diff: jena-arq/src/test/java/org/apache/jena/riot/TestJsonLDReader.java --- @@ -0,0 +1,81 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] jena issue #203: JsonLDReader: possibility to override the @context

2017-01-08 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/jena/pull/203 This is good stuff-- thanks for taking a problem and turning it into an improvement! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] jena pull request #203: JsonLDReader: possibility to override the @context

2017-01-08 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/203#discussion_r95094991 --- Diff: jena-arq/src/test/java/org/apache/jena/riot/TestJsonLDReader.java --- @@ -0,0 +1,81 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] jena pull request #203: JsonLDReader: possibility to override the @context

2017-01-08 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/203#discussion_r95094927 --- Diff: jena-arq/src/test/java/org/apache/jena/riot/TestJsonLDReader.java --- @@ -0,0 +1,81 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] jena pull request #203: JsonLDReader: possibility to override the @context

2017-01-08 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/203#discussion_r95094877 --- Diff: jena-arq/src/test/java/org/apache/jena/riot/TestJsonLDReader.java --- @@ -0,0 +1,81 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] jena pull request #203: JsonLDReader: possibility to override the @context

2017-01-08 Thread fpservant
GitHub user fpservant opened a pull request: https://github.com/apache/jena/pull/203 JsonLDReader: possibility to override the @context following a question on the user list: "[Overriding @context when loading JSON-LD](http://jena.markmail.org/message/uf3zkzybv7gb4crt)", a way

[GitHub] jena pull request #:

2017-01-08 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20400578 In jena-arq/src/test/java/org/apache/jena/sparql/core/pergraph/MultithreadingTest.java: In

[GitHub] jena pull request #:

2017-01-08 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20400171 In jena-arq/src/test/java/org/apache/jena/sparql/core/pergraph/MultithreadingTest.java: In

[GitHub] jena pull request #:

2017-01-08 Thread afs
Github user afs commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20400054 In jena-arq/src/test/java/org/apache/jena/sparql/core/pergraph/MultithreadingTest.java: In

[GitHub] jena pull request #202: Formatting rampage

2017-01-08 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/202#discussion_r95081992 --- Diff: jena-base/src/main/java/org/apache/jena/atlas/lib/CollectionUtils.java --- @@ -22,12 +22,7 @@ public class CollectionUtils { -

[GitHub] jena issue #202: Formatting rampage

2017-01-08 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/jena/pull/202 Autogen code is now back where it started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] jena pull request #202: Formatting rampage

2017-01-08 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/202#discussion_r95081789 --- Diff: jena-base/src/main/java/org/apache/jena/atlas/lib/SetUtils.java --- @@ -44,13 +42,26 @@ private SetUtils() {} return

[GitHub] jena pull request #:

2017-01-08 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399785 In jena-arq/src/test/java/org/apache/jena/sparql/core/pergraph/MultithreadingTest.java: In

[GitHub] jena pull request #:

2017-01-08 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399635 In jena-arq/src/main/java/org/apache/jena/sparql/core/DatasetGraphPerGraphLocking.java: In

[GitHub] jena pull request #:

2017-01-08 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399629 In jena-arq/src/test/java/org/apache/jena/sparql/core/pergraph/MultithreadingTest.java: In

[GitHub] jena pull request #:

2017-01-08 Thread afs
Github user afs commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399562 In jena-arq/src/main/java/org/apache/jena/sparql/core/DatasetGraphPerGraphLocking.java: In

[jira] [Commented] (JENA-1274) Support a writer-per-graph in-memory dataset

2017-01-08 Thread A. Soroka (JIRA)
[ https://issues.apache.org/jira/browse/JENA-1274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15809329#comment-15809329 ] A. Soroka commented on JENA-1274: - The background is the project that originally led me to Jena-- Fedora

[GitHub] jena pull request #:

2017-01-08 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399506 In jena-arq/src/main/java/org/apache/jena/sparql/core/DatasetGraphPerGraphLocking.java: In

[GitHub] jena pull request #:

2017-01-08 Thread afs
Github user afs commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399491 In jena-arq/src/main/java/org/apache/jena/sparql/core/DatasetGraphPerGraphLocking.java: In

[jira] [Commented] (JENA-1274) Support a writer-per-graph in-memory dataset

2017-01-08 Thread Andy Seaborne (JIRA)
[ https://issues.apache.org/jira/browse/JENA-1274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15809288#comment-15809288 ] Andy Seaborne commented on JENA-1274: - Looks interesting! GH does not make discussions of branches as

[GitHub] jena pull request #:

2017-01-08 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399405 In jena-arq/src/test/java/org/apache/jena/sparql/core/pergraph/MultithreadingTest.java: In

[GitHub] jena pull request #:

2017-01-08 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399404 In jena-arq/src/main/java/org/apache/jena/sparql/core/DatasetGraphPerGraphLocking.java: In

[GitHub] jena pull request #:

2017-01-08 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/commit/f00f0e0c7d7e9952a3f3f840fe6a611d4043e5a6#commitcomment-20399392 In jena-arq/src/main/java/org/apache/jena/sparql/core/DatasetGraphPerGraphLocking.java: In

[GitHub] jena pull request #:

2017-01-08 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399391 In jena-arq/src/main/java/org/apache/jena/sparql/core/DatasetGraphPerGraphLocking.java: In

[GitHub] jena pull request #:

2017-01-08 Thread afs
Github user afs commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399281 In jena-arq/src/main/java/org/apache/jena/sparql/core/DatasetGraphPerGraphLocking.java: In

[GitHub] jena pull request #:

2017-01-08 Thread afs
Github user afs commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399247 In jena-arq/src/main/java/org/apache/jena/sparql/core/DatasetGraphPerGraphLocking.java: In

[GitHub] jena pull request #:

2017-01-08 Thread afs
Github user afs commented on the pull request: https://github.com/apache/jena/commit/8601cd3028c0e57dbc805efc5d54d7b9ee272c3f#commitcomment-20399234 In jena-arq/src/test/java/org/apache/jena/sparql/core/pergraph/MultithreadingTest.java: In

Re: classloader and testing

2017-01-08 Thread Andy Seaborne
For testing with Fuseki, then the embedded version is useful as used in the integration tests - it does not need any static resources (web pages), just a dataset and a name for the URL. e.g. use in: