[GitHub] incubator-rya issue #274: Added giraph profile to resolve an incompatible li...

2018-02-16 Thread joshelser
Github user joshelser commented on the issue: https://github.com/apache/incubator-rya/pull/274 Hrm, maybe worth it to send a note to the Giraph mailing lists? If this really is an LGPL dependency, then Apache Giraph shouldn't be depending on it either. ---

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-11-02 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r86241465 --- Diff: NOTICE --- @@ -4,3 +4,25 @@ Copyright 2016 The Apache Software Foundation This product includes software developed at The Apache

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-11-02 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r86242245 --- Diff: extras/rya.jena.sesame/NOTICE --- @@ -0,0 +1,11 @@ +Apache Jena - ARQ module +Copyright 2011, 2012 The Apache Software Foundation

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-11-02 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r86241962 --- Diff: extras/rya.jena.sesame/Copyright.txt --- @@ -0,0 +1,25 @@ + + (c) Copyright 2009 Talis Information Ltd. + All rights

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-11-02 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r86241601 --- Diff: NOTICE --- @@ -4,3 +4,25 @@ Copyright 2016 The Apache Software Foundation This product includes software developed at The Apache

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-11-02 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r86241337 --- Diff: LICENSE --- @@ -200,3 +200,42 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-11-02 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r86243323 --- Diff: extras/rya.jena.sesame/src/main/java/org/apache/rya/jena/jenasesame/impl/Convert.java --- @@ -0,0 +1,209 @@ +/* --- End diff

[GitHub] incubator-rya issue #115: RYA-200 Removing findbugs:jsr305 Dependency

2016-10-18 Thread joshelser
Github user joshelser commented on the issue: https://github.com/apache/incubator-rya/pull/115 Not sure about ParametersAreNonNullByDefault, curiously, it's mentioned https://github.com/stephenc/findbugs-annotations/blob/master/src/spec/javadoc/edu/umd/cs/findbugs/annotations

[GitHub] incubator-rya issue #115: RYA-200 Removing findbugs:jsr305 Dependency

2016-10-18 Thread joshelser
Github user joshelser commented on the issue: https://github.com/apache/incubator-rya/pull/115 ThreadSafe and Immutable are available in https://github.com/stephenc/jcip-annotations --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-rya issue #101: RYA-177 adding optional geoindexing profile

2016-10-11 Thread joshelser
Github user joshelser commented on the issue: https://github.com/apache/incubator-rya/pull/101 Looks reasonable enough. I'll try to pull down the changes and take a look tmrw with fresh eyes. --- If your project is set up for it, you can reply to this email and have your reply