Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Kamila Molina Orellana
Hi, Sorry for causing the confusion. I mean to include a general subject since I think commons would benefit from the results. However, I pointed out in the content that the work'd be done over commons-rdf. Again, sorry for that. :D Regards, ~Kamila. On Thu, Mar 8, 2018 at 8:23 PM, G

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Gilles
On Fri, 9 Mar 2018 01:47:01 +0100, Bernd Eckenfels wrote: Hello, any experience with compiling a JAR with old Java and only adding the module-info.jar with a new class Version? Would that allow to avoid the Need for Multi-Release JARs? (of Course it makes a ugly toolchain). IIUC, there must be

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Ralph Goers
> On Mar 8, 2018, at 5:47 PM, Bernd Eckenfels wrote: > > Hello, > > any experience with compiling a JAR with old Java and only adding the > module-info.jar with a new class Version? Would that allow to avoid the Need > for Multi-Release JARs? (of Course it makes a ugly toolchain). Log4j 2 h

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Bernd Eckenfels
Hello, any experience with compiling a JAR with old Java and only adding the module-info.jar with a new class Version? Would that allow to avoid the Need for Multi-Release JARs? (of Course it makes a ugly toolchain). Gruss Bernd -- http://bernd.eckenfels.net

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Gilles
On Thu, 8 Mar 2018 17:19:05 -0700, Ralph Goers wrote: On Mar 8, 2018, at 5:17 PM, Ralph Goers wrote: On Mar 8, 2018, at 4:08 PM, Gilles wrote: On Thu, 8 Mar 2018 15:09:18 -0700, Ralph Goers wrote: On Mar 8, 2018, at 11:06 AM, Gilles wrote: On Thu, 8 Mar 2018 11:01:08 -0700, Gary Greg

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Gary Gregory
That seems like a good pragmatic way about it. Gary On Thu, Mar 8, 2018, 16:08 Gilles wrote: > On Thu, 8 Mar 2018 15:09:18 -0700, Ralph Goers wrote: > >> On Mar 8, 2018, at 11:06 AM, Gilles > >> wrote: > >> > >> On Thu, 8 Mar 2018 11:01:08 -0700, Gary Gregory wrote: > >>> On Thu, Mar 8, 2018 a

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Gary Gregory
I almost pooped myself on that one Ralph :-) Good one! Gary On Thu, Mar 8, 2018, 17:19 Ralph Goers wrote: > > > > On Mar 8, 2018, at 5:17 PM, Ralph Goers > wrote: > > > > > > > >> On Mar 8, 2018, at 4:08 PM, Gilles > wrote: > >> > >> On Thu, 8 Mar 2018 15:09:18 -0700, Ralph Goers wrote: > >>>

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Gilles
On Thu, 8 Mar 2018 17:17:40 -0700, Ralph Goers wrote: On Mar 8, 2018, at 4:08 PM, Gilles wrote: On Thu, 8 Mar 2018 15:09:18 -0700, Ralph Goers wrote: On Mar 8, 2018, at 11:06 AM, Gilles wrote: On Thu, 8 Mar 2018 11:01:08 -0700, Gary Gregory wrote: On Thu, Mar 8, 2018 at 10:58 AM, Gilles

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Ralph Goers
> On Mar 8, 2018, at 5:17 PM, Ralph Goers wrote: > > > >> On Mar 8, 2018, at 4:08 PM, Gilles wrote: >> >> On Thu, 8 Mar 2018 15:09:18 -0700, Ralph Goers wrote: On Mar 8, 2018, at 11:06 AM, Gilles wrote: On Thu, 8 Mar 2018 11:01:08 -0700, Gary Gregory wrote: > On Thu, Ma

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Ralph Goers
> On Mar 8, 2018, at 4:08 PM, Gilles wrote: > > On Thu, 8 Mar 2018 15:09:18 -0700, Ralph Goers wrote: >>> On Mar 8, 2018, at 11:06 AM, Gilles wrote: >>> >>> On Thu, 8 Mar 2018 11:01:08 -0700, Gary Gregory wrote: On Thu, Mar 8, 2018 at 10:58 AM, Gilles wrote: > On Thu, 8 M

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Gilles
On Thu, 8 Mar 2018 15:09:18 -0700, Ralph Goers wrote: On Mar 8, 2018, at 11:06 AM, Gilles wrote: On Thu, 8 Mar 2018 11:01:08 -0700, Gary Gregory wrote: On Thu, Mar 8, 2018 at 10:58 AM, Gilles wrote: On Thu, 8 Mar 2018 10:48:28 -0700, Gary Gregory wrote: On Thu, Mar 8, 2018 at 10:29 AM, G

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Ralph Goers
> On Mar 8, 2018, at 11:06 AM, Gilles wrote: > > On Thu, 8 Mar 2018 11:01:08 -0700, Gary Gregory wrote: >> On Thu, Mar 8, 2018 at 10:58 AM, Gilles >> wrote: >> >>> On Thu, 8 Mar 2018 10:48:28 -0700, Gary Gregory wrote: >>> On Thu, Mar 8, 2018 at 10:29 AM, Gilles wrote:

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Gilles
and the marks of success are fundamentally different than for many other contributions. Gilles has rightly pointed out that this is about Commons RDF and that is all. Kamila unfortunately didn't make that clear in the subject line of the thread, but that was just a slip of the keyboa

Re: Prepare commons RDF to JDK 9 Was: Prepare commons to JDK 9

2018-03-08 Thread ajs6f
> On Mar 8, 2018, at 12:48 PM, Gary Gregory wrote: > On Thu, Mar 8, 2018 at 10:29 AM, Gilles > wrote: >> Then these modules can define "module-info" files, and an actual build will >> prove that the dependencies are as expected. >> > As Ralph as pointed out, you cannot generate a module-info f

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Gary Gregory
>> Hi, this is Adam Soroka (prospective mentor for this project). I'm >>>>> sorry >>>>> to have been absent from this conversation so far (been very busy this >>>>> week) but Gilles has said much of what I would have said, so thanks >>>

Re: Prepare commons RDF to JDK 9

2018-03-08 Thread Gilles
o far (been very busy this week) but Gilles has said much of what I would have said, so thanks Gilles! I would emphasize a couple of points: This is a GSoC project. The expectations and the marks of success are fundamentally different than for many other contributions. Gilles has rightly

Re: Prepare commons RDF to JDK 9 Was: Prepare commons to JDK 9

2018-03-08 Thread Gary Gregory
>>> Hi, this is Adam Soroka (prospective mentor for this project). I'm sorry >>> to have been absent from this conversation so far (been very busy this >>> week) but Gilles has said much of what I would have said, so thanks >>> Gilles! >>> >>

Re: Prepare commons RDF to JDK 9 Was: Prepare commons to JDK 9

2018-03-08 Thread Gary Gregory
gt; Yes. That's the point. That's perfectly fine. Kamila will find out which > break, and how, and what options we have to deal with them, and when we > might need to go upstream and come back, and she will document that, and > she will do what work can be done now on those problems

Re: Prepare commons RDF to JDK 9 Was: Prepare commons to JDK 9

2018-03-08 Thread Gilles
I would emphasize a couple of points: This is a GSoC project. The expectations and the marks of success are fundamentally different than for many other contributions. Gilles has rightly pointed out that this is about Commons RDF and that is all. Kamila unfortunately didn't make that clear in

Re: Prepare commons RDF to JDK 9 Was: Prepare commons to JDK 9

2018-03-08 Thread ajs6f
m, and when we might need to go upstream and come back, and she will document that, and she will do what work can be done now on those problems. If Commons RDF does not release MR JARs as part of the GSoC project, but we do understand specifically and in detail why and what the blockers are, the

Re: Prepare commons RDF to JDK 9 Was: Prepare commons to JDK 9

2018-03-08 Thread Gary Gregory
s Gilles! > > I would emphasize a couple of points: > > This is a GSoC project. The expectations and the marks of success are > fundamentally different than for many other contributions. > > Gilles has rightly pointed out that this is about Commons RDF and that is > all. Kami

Prepare commons RDF to JDK 9 Was: Prepare commons to JDK 9

2018-03-08 Thread ajs6f
damentally different than for many other contributions. Gilles has rightly pointed out that this is about Commons RDF and that is all. Kamila unfortunately didn't make that clear in the subject line of the thread, but that was just a slip of the keyboard. It's not about any other

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2018-02-14 Thread stain
Github user stain commented on the issue: https://github.com/apache/commons-rdf/pull/43 You may have better ideas on how to do something like [ParserConfigImpl](https://github.com/apache/commons-rdf/blob/fluent-parser/commons-rdf-api/src/main/java/org/apache/commons/rdf/api/io

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2018-02-14 Thread stain
Github user stain commented on the issue: https://github.com/apache/commons-rdf/pull/43 Thanks, @ajs6f ! I found some old code where I had tried to make a fluent interface.. I managed to update it to the current master and sorted some issues. I pushed it to the

[GitHub] commons-rdf pull request #49: Cleanup for FindBugs and PMD warnings in -simp...

2018-02-14 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/commons-rdf/pull/49#discussion_r168275645 --- Diff: commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/experimental/AbstractRDFParser.java --- @@ -58,8 +60,12 @@ */ public

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2018-02-14 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/commons-rdf/pull/43 Okay, I'll get to work on factoring out a (serializable) factory. Thanks for the discussion everyone! I think the parser types will be much stronger fo

[GitHub] commons-rdf pull request #49: Cleanup for FindBugs and PMD warnings in -simp...

2018-02-14 Thread stain
Github user stain commented on a diff in the pull request: https://github.com/apache/commons-rdf/pull/49#discussion_r168274256 --- Diff: commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/experimental/AbstractRDFParser.java --- @@ -58,8 +60,12 @@ */ public

[GitHub] commons-rdf pull request #43: COMMONSRDF-49: Make AbstractRDFParser serializ...

2018-02-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-rdf/pull/43 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2018-02-14 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/commons-rdf/pull/43 Hm... if @afs and @stain are both feeling reluctant to go this way... I would still be happy to merge it as-is and then work forward to the fluent-ish factory design (since @ansell has given a

[GitHub] commons-rdf pull request #49: Cleanup for FindBugs and PMD warnings in -simp...

2018-02-14 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/commons-rdf/pull/49#discussion_r168260775 --- Diff: commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/experimental/AbstractRDFParser.java --- @@ -58,8 +60,12 @@ */ public

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2018-02-12 Thread stain
Github user stain commented on the issue: https://github.com/apache/commons-rdf/pull/43 Picking up this - @ajs6f do you still think we should proceed along those lines? I am also reluctant to making the abstract factory (builder) serializable, but I can see the reasoning

[GitHub] commons-rdf pull request #48: Cleanup in commons-rdf-rdf4j to close PMD and ...

2018-02-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-rdf/pull/48 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rdf pull request #50: Cleanup for PMD warnings in -rdf-api

2018-02-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-rdf/pull/50 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rdf pull request #49: Cleanup for FindBugs and PMD warnings in -simp...

2018-02-12 Thread stain
Github user stain commented on a diff in the pull request: https://github.com/apache/commons-rdf/pull/49#discussion_r167732171 --- Diff: commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/experimental/AbstractRDFParser.java --- @@ -58,8 +60,12 @@ */ public

[GitHub] commons-rdf pull request #52: Adding convenient Dataset and DatasetGraph con...

2018-02-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-rdf/pull/52 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rdf issue #48: Cleanup in commons-rdf-rdf4j to close PMD and FindBug...

2018-01-30 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/commons-rdf/pull/48 @wikier I've got a triple of PRs here to clean up warnings, as we heard about last release. Is there a problem with merging them? Is there something I should do to help them merge-able? T

[GitHub] commons-rdf pull request #47: COMMONSRDF-67: Make commons-rdf buildable on J...

2018-01-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-rdf/pull/47 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rdf pull request #45: COMMONSRDF-71: Update Javadocs link for RDF4J

2018-01-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-rdf/pull/45 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rdf issue #45: COMMONSRDF-71: Update Javadocs link for RDF4J

2018-01-29 Thread wikier
Github user wikier commented on the issue: https://github.com/apache/commons-rdf/pull/45 Good catch. Maybe at some point we just want to have http://docs.rdf4j.org/javadoc/latest/ but LGTM. --- - To unsubscribe, e

[GitHub] commons-rdf pull request #46: Add .DS_Store to .gitignore (Mac OS X system f...

2018-01-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-rdf/pull/46 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rdf issue #46: Add .DS_Store to .gitignore (Mac OS X system file)

2018-01-29 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/commons-rdf/pull/46 Is there any problem with merging this? Is there anything I can do to move it along? --- - To unsubscribe, e-mail: dev

[GitHub] commons-rdf pull request #52: Adding convenient Dataset and DatasetGraph con...

2018-01-29 Thread ajs6f
GitHub user ajs6f opened a pull request: https://github.com/apache/commons-rdf/pull/52 Adding convenient Dataset and DatasetGraph conversions PR for COMMONSRDF-74. You can merge this pull request into a Git repository by running: $ git pull https://github.com/ajs6f/commons-rdf

Fwd: [ANNOUNCE] Apache Commons RDF 0.5.0 released

2017-12-23 Thread Sergio Fernández
The Apache Commons RDF team is pleased to announce the release of Apache Commons RDF 0.5.0. Commons RDF aims to provide a common library for RDF 1.1 that could be implemented by systems on the Java Virtual Machine. The Apache Commons RDF distribution includes bindings for different

[ANNOUNCE] Apache Commons RDF 0.5.0 released

2017-12-23 Thread Sergio Fernández
The Apache Commons RDF team is pleased to announce the release of Apache Commons RDF 0.5.0. Commons RDF aims to provide a common library for RDF 1.1 that could be implemented by systems on the Java Virtual Machine. The Apache Commons RDF distribution includes bindings for different

[GitHub] commons-rdf pull request #51: 0.5.0 Release Candidate

2017-12-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-rdf/pull/51 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rdf issue #51: 0.5.0 Release Candidate

2017-12-22 Thread wikier
Github user wikier commented on the issue: https://github.com/apache/commons-rdf/pull/51 VOTE passed, see: https://lists.apache.org/thread.html/c7440213bd41f9839fdce7aab957e686a954affdf5378ff40ea23dbe@%3Cdev.commons.apache.org%3E

[GitHub] commons-rdf pull request #51: 0.5.0 Release Candidate

2017-12-22 Thread wikier
GitHub user wikier opened a pull request: https://github.com/apache/commons-rdf/pull/51 0.5.0 Release Candidate You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/commons-rdf 0.5.0-RC Alternatively you can review and apply

[RESULT] [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-22 Thread Sergio Fernández
, Sergio On Thu, Dec 7, 2017 at 8:51 PM, Sergio Fernández wrote: > Hi, > > once we addressed most of the issues from RC1 and RC2, I'd like to propose > to release Apache Commons RDF 0.5.0 based on RC. > > Apache Commons RDF aims to provide a common Java API for RDF 1.1 gra

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-22 Thread Sergio Fernández
fix as we need > >>cannot create a model twice...) > >> > >> * commons-df-api: > >>- Initialization of org.apache.commons.rdf.api.RDFSyntax accesses > >>class > >> org.apache.commons.rdf.api.W3CRDFSyntax, which isn't initialized

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-21 Thread Bruno P. Kinoshita
> * commons-df-api: >>    - Initialization of org.apache.commons.rdf.api.RDFSyntax accesses >>    class >> org.apache.commons.rdf.api.W3CRDFSyntax, which isn't initialized yet >> (this one seems easy to reproduce. Always returns null?) >> >> * commons-rdf-simp

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-21 Thread ajs6f
ix as we need >>cannot create a model twice...) >> >> * commons-df-api: >>- Initialization of org.apache.commons.rdf.api.RDFSyntax accesses >>class >> org.apache.commons.rdf.api.W3CRDFSyntax, which is

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-21 Thread Jörg Schaible
itialized yet > (this one seems easy to reproduce. Always returns null?) > > * commons-rdf-simple: > - org.apache.commons.rdf.simple.DatasetGraphView.unionOrNamedGraph() > has Optional return type and returns explicit null (shouldn't it be > Optional.empty() or someth

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-21 Thread Jörg Schaible
Hi, Am Thu, 07 Dec 2017 20:51:57 -0800 schrieb Sergio Fernández: > Hi, > > once we addressed most of the issues from RC1 and RC2, I'd like to > propose to release Apache Commons RDF 0.5.0 based on RC. > > Apache Commons RDF aims to provide a common Java API for RDF 1.

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-20 Thread Sergio Fernández
family: >>> "windows" >>> > >>> > The code needs clean up next, for example "Avoid modifiers which are >>> > implied by the context". This means for example, that method decls in >>> > interfaces do not need to be '

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-17 Thread Sergio Fernández
#x27;, it is implied. >> > >> > You should also provide a site with the RC, either a full site on your >> > home.apache.org folder or as a zip file. >> > >> > +1 >> > >> > Gary >> > >> > On Thu, Dec 7, 2017 at 9:51 PM, S

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-14 Thread Sergio Fernández
is implied. > > > > You should also provide a site with the RC, either a full site on your > > home.apache.org folder or as a zip file. > > > > +1 > > > > Gary > > > > On Thu, Dec 7, 2017 at 9:51 PM, Sergio Fernández > > wrote: > > > &

[GitHub] commons-rdf pull request #50: Cleanup for PMD warnings in -rdf-api

2017-12-14 Thread ajs6f
GitHub user ajs6f opened a pull request: https://github.com/apache/commons-rdf/pull/50 Cleanup for PMD warnings in -rdf-api You can merge this pull request into a Git repository by running: $ git pull https://github.com/ajs6f/commons-rdf CleanupAPI Alternatively you can

[GitHub] commons-rdf pull request #49: Cleanup for FindBugs and PMD warnings in -simp...

2017-12-14 Thread ajs6f
GitHub user ajs6f opened a pull request: https://github.com/apache/commons-rdf/pull/49 Cleanup for FindBugs and PMD warnings in -simple and -jena You can merge this pull request into a Git repository by running: $ git pull https://github.com/ajs6f/commons-rdf

[GitHub] commons-rdf pull request #48: Cleanup in commons-rdf-rdf4j to close PMD and ...

2017-12-14 Thread ajs6f
GitHub user ajs6f opened a pull request: https://github.com/apache/commons-rdf/pull/48 Cleanup in commons-rdf-rdf4j to close PMD and FindBugs warnings You can merge this pull request into a Git repository by running: $ git pull https://github.com/ajs6f/commons-rdf

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-14 Thread Gary Gregory
not need to be 'public', it is implied. > > You should also provide a site with the RC, either a full site on your > home.apache.org folder or as a zip file. > > +1 > > Gary > > On Thu, Dec 7, 2017 at 9:51 PM, Sergio Fernández > wrote: > >> Hi, &

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-14 Thread Gary Gregory
wrote: > Hi, > > once we addressed most of the issues from RC1 and RC2, I'd like to propose > to release Apache Commons RDF 0.5.0 based on RC. > > Apache Commons RDF aims to provide a common Java API for RDF 1.1 graphs and > datasets. API bindings in Commons RDF 0.5.0 i

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-14 Thread Gary Gregory
Starting to review but this is an unusual distro in https://dist.apache.org/repos/dist/dev/commons/rdf/apache- commons-rdf-0.5.0-RC3/ There are no release-notes.txt file and no bin zip file. Gary On Thu, Dec 7, 2017 at 9:51 PM, Sergio Fernández wrote: > Hi, > > once we addressed mo

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-13 Thread Bruno P. Kinoshita
zed yet (this one seems easy to reproduce. Always returns null?) * commons-rdf-simple: - org.apache.commons.rdf.simple.DatasetGraphView.unionOrNamedGraph() has Optional return type and returns explicit null (shouldn't it be Optional.empty() or something similar?) Are any of

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-12 Thread Sergio Fernández
9T21:20:00+01:00 > [INFO] Final Memory: 63M/662M > [INFO] > > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-site-plugin:3.6:site (default-site) on > project commons-rdf-parent: Error generating > maven-checkstyle

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-12 Thread ajs6f
t in Apache Karaf 4.1.2 > > Checked that NOTICE and LICENSE files are contained in each binary artifact. > > Thanks, > Aaron Coburn > acob...@apache.org > > > On 12/7/17, 11:52 PM, "Sergio Fernández" wrote: > >Hi, > >once we addressed most

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-12 Thread Aaron Coburn
ssues from RC1 and RC2, I'd like to propose to release Apache Commons RDF 0.5.0 based on RC. Apache Commons RDF aims to provide a common Java API for RDF 1.1 graphs and datasets. API bindings in Commons RDF 0.5.0 include Apache Jena, Eclipse RDF4J, JSON-LD Java as well a

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-09 Thread Oliver Heger
to execute goal org.apache.maven.plugins:maven-site-plugin:3.6:site (default-site) on project commons-rdf-parent: Error generating maven-checkstyle-plugin:2.12.1:checkstyle-aggregate: Failed during checkstyle configuration: cannot initialize module TreeWalker - Property

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-07 Thread Sergio Fernández
my +1 (non-binding) On Thu, Dec 7, 2017 at 8:51 PM, Sergio Fernández wrote: > Hi, > > once we addressed most of the issues from RC1 and RC2, I'd like to propose > to release Apache Commons RDF 0.5.0 based on RC. > > Apache Commons RDF aims to provide a common Java API f

[VOTE] Release Apache Commons RDF 0.5.0 based on RC3

2017-12-07 Thread Sergio Fernández
Hi, once we addressed most of the issues from RC1 and RC2, I'd like to propose to release Apache Commons RDF 0.5.0 based on RC. Apache Commons RDF aims to provide a common Java API for RDF 1.1 graphs and datasets. API bindings in Commons RDF 0.5.0 include Apache Jena, Eclipse RDF4J, JSON-LD

[CANCELLED} [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-12-06 Thread Sergio Fernández
#x27;s stuck. It's fine to get -1s, but at least something to move forward. > > Thanks. > > > > On Nov 22, 2017 17:52, "Sergio Fernández" wrote: > > > >> Hi Oliver, > >> > >> thanks for the feedback on RC2. Please, find my answers in

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-12-06 Thread ajs6f
any, vote for this RC. Because >> it's stuck. It's fine to get -1s, but at least something to move forward. >> Thanks. >> >> On Nov 22, 2017 17:52, "Sergio Fernández" wrote: >> >>> Hi Oliver, >>> >>> thanks for the

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-12-06 Thread Stian Soiland-Reyes
Fernández" wrote: > >> Hi Oliver, >> >> thanks for the feedback on RC2. Please, find my answers inline. >> >> On Wed, Nov 22, 2017 at 1:25 PM, Oliver Heger < >> oliver.he...@oliver-heger.de> wrote: >>> >>> [ERROR] Failed to execute

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-12-04 Thread Sergio Fernández
OR] Failed to execute goal >> org.apache.maven.plugins:maven-site-plugin:3.6:site (default-cli) on >> project commons-rdf-parent: Error generating >> maven-checkstyle-plugin:2.12.1:checkstyle-aggregate: Failed during >> checkstyle configuration: cannot initialize module Tree

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-26 Thread Sergio Fernández
se, find my answers inline. > > On Wed, Nov 22, 2017 at 1:25 PM, Oliver Heger < > oliver.he...@oliver-heger.de> wrote: >> >> [ERROR] Failed to execute goal >> org.apache.maven.plugins:maven-site-plugin:3.6:site (default-cli) on >> project commons-rdf-parent

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-22 Thread Sergio Fernández
Hi Oliver, thanks for the feedback on RC2. Please, find my answers inline. On Wed, Nov 22, 2017 at 1:25 PM, Oliver Heger wrote: > > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-site-plugin:3.6:site (default-cli) on > project commons-rdf-parent: Error genera

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-22 Thread Oliver Heger
Hi Sergio, build works fine with Java 1.8 on Windows 10. However, the site build fails for me with the following error: [INFO] [INFO] Reactor Summary: [INFO] [INFO] Commons RDF

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-20 Thread Sergio Fernández
ives.apache.org/mod_mbox/commons-commits/201 >> 711.mbox/browser > > > Thanks for the pointer, sebb! > > Then, I'd like to update the VOTE to include the following artifacts have > been staged for Apache Commons RDF 0.5.0 RC2: > > commons-rdf-rdf4j-0.5.0-tests.jar.as

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-20 Thread Sergio Fernández
o include the following artifacts have been staged for Apache Commons RDF 0.5.0 RC2: commons-rdf-rdf4j-0.5.0-tests.jar.asc (SHA1: 04605bc88577af60fa0762ca279be80a57a85958) commons-rdf-rdf4j-0.5.0.pom (SHA1: 1cdc74b7205fa06531bd59e1ee24f1d15999ab1b) commons-rdf-rdf4j-0.5.0-javadoc.jar.asc (SHA1: 8f2f0

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-20 Thread sebb
On 20 November 2017 at 21:36, Sergio Fernández wrote: > Hi, > > On Mon, Nov 20, 2017 at 10:54 AM, Gary Gregory > wrote: > >> On Sun, Nov 19, 2017 at 3:08 PM, Sergio Fernández >> wrote: >> > Apache Commons RDF 0.5.0 RC2 is available for review at (r23205): >

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-20 Thread Sergio Fernández
Hi, On Mon, Nov 20, 2017 at 10:54 AM, Gary Gregory wrote: > On Sun, Nov 19, 2017 at 3:08 PM, Sergio Fernández > wrote: > > Apache Commons RDF 0.5.0 RC2 is available for review at (r23205): > > > > https://dist.apache.org/repos/dist/dev/commons/rdf/apache-co > >

[GitHub] commons-rdf pull request #47: COMMONSRDF-67

2017-11-20 Thread acoburn
GitHub user acoburn opened a pull request: https://github.com/apache/commons-rdf/pull/47 COMMONSRDF-67 The current japicmp maven plugin requires a number of javaee libraries that are not available on JDK 9. Until this is fixed in the upstream project, this adds the required

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-20 Thread Gary Gregory
On Sun, Nov 19, 2017 at 3:08 PM, Sergio Fernández wrote: > Hi, > > once we addressed most of the issues from RC1, I'd like to propose to > release Apache Commons RDF 0.5.0 based on RC2. > > Apache Commons RDF aims to provide a common Java API for RDF 1.1 graphs and >

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-20 Thread ajs6f
> Cheers, >Bruno > > > > > >From: Sergio Fernández >To: Commons Developers List >Sent: Monday, 20 November 2017 11:08 AM >Subject: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2 > >

[GitHub] commons-rdf pull request #46: Add .DS_Store to .gitignore (Mac OS X system f...

2017-11-20 Thread ajs6f
GitHub user ajs6f opened a pull request: https://github.com/apache/commons-rdf/pull/46 Add .DS_Store to .gitignore (Mac OS X system file) You can merge this pull request into a Git repository by running: $ git pull https://github.com/ajs6f/commons-rdf patch-1 Alternatively

[GitHub] commons-rdf pull request #45: COMMONSRDF-71

2017-11-20 Thread acoburn
GitHub user acoburn opened a pull request: https://github.com/apache/commons-rdf/pull/45 COMMONSRDF-71 Updates location of RDF4J javadocs link You can merge this pull request into a Git repository by running: $ git pull https://github.com/acoburn/commons-rdf COMMONSRDF-71

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-20 Thread Aaron Coburn
To: Commons Developers List Sent: Monday, 20 November 2017 11:08 AM Subject: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2 Hi, once we addressed most of the issues from RC1, I'd like to propose to release Apache Commons RDF 0.5.0

Re: [VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-20 Thread Bruno P. Kinoshita
s warnings in the reports if preferable. Checked signatures of jars in Maven repository, all good. Thanks for preparing this new release. Cheers, Bruno From: Sergio Fernández To: Commons Developers List Sent: Monday, 20 November 2017 11:08 AM Subject: [VOTE] R

[VOTE] Release Apache Commons RDF 0.5.0 based on RC2

2017-11-19 Thread Sergio Fernández
Hi, once we addressed most of the issues from RC1, I'd like to propose to release Apache Commons RDF 0.5.0 based on RC2. Apache Commons RDF aims to provide a common Java API for RDF 1.1 graphs and datasets. API bindings in Commons RDF 0.5.0 include Apache Jena, Eclipse RDF4J, JSON-LD Ja

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2017-11-19 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/commons-rdf/pull/43 @wikier My sense is that the right move here immediately is to merge this PR and then open a ticket to factor config (and a builder therefor) out of `AbstractRDFParser`. (A ticket I will be happy

[GitHub] commons-rdf pull request #43: COMMONSRDF-49: Make AbstractRDFParser serializ...

2017-11-15 Thread ansell
Github user ansell commented on a diff in the pull request: https://github.com/apache/commons-rdf/pull/43#discussion_r151267617 --- Diff: commons-rdf-simple/src/main/java/org/apache/commons/rdf/simple/experimental/AbstractRDFParser.java --- @@ -200,19 +200,19

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2017-11-15 Thread ansell
Github user ansell commented on the issue: https://github.com/apache/commons-rdf/pull/43 Having ``Optional`` fields isn't impossible to serialise (as I said erroneously in the referenced comment), as you could always write custom serialise/deserialise code to support it, but it

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2017-11-15 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/commons-rdf/pull/43 👍 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2017-11-15 Thread wikier
Github user wikier commented on the issue: https://github.com/apache/commons-rdf/pull/43 Having a little understanding on the concrete use case behind this PR, generally speaking I normally prefer having the factories/builders implementing `Serializable`. So in this concrete case I

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2017-11-15 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/commons-rdf/pull/43 I'd like to get @wikier an answer to his question. It sounds like we are _not_ comfortable merging this for `RC2` and he should go ahead without it, correct? If the consensus is

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2017-11-15 Thread stain
Github user stain commented on the issue: https://github.com/apache/commons-rdf/pull/43 Agree with @afs that only the builder-factory bit should be serialized - obviously the actual parser which may be in progress of parsing is tricky to serialize. So we would need to perhaps clean

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2017-11-14 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/commons-rdf/pull/43 So the question is whether what is here now is a step forward. I think it is, and I would want to do what is here whether or not we go onto a builder factoring. I intentionally did not add

[GitHub] commons-rdf issue #43: COMMONSRDF-49: Make AbstractRDFParser serializable

2017-11-13 Thread afs
Github user afs commented on the issue: https://github.com/apache/commons-rdf/pull/43 All I did was point out that it isn't logically serializable at all points in its usage. My suggestion is that there is design with a builder (serializable) and instance (not seriali

[GitHub] commons-rdf pull request #44: COMMONSRDF-70: Upgrade Jena version to 3.5.0

2017-11-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-rdf/pull/44 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rdf issue #44: COMMONSRDF-70: Upgrade Jena version to 3.5.0

2017-11-13 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/commons-rdf/pull/44 👍 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h

[GitHub] commons-rdf issue #44: COMMONSRDF-70: Upgrade Jena version to 3.5.0

2017-11-13 Thread wikier
Github user wikier commented on the issue: https://github.com/apache/commons-rdf/pull/44 Now I think this should definitively go in `0.5.0-RC2`. --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For

  1   2   3   4   5   >