You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pr@jena.apache.org by "afs (via GitHub)" <gi...@apache.org> on 2023/06/21 14:18:59 UTC

[GitHub] [jena] afs opened a new pull request, #1915: Don't have a shaded Google Guava artifact. Use alternative class where possible.

afs opened a new pull request, #1915:
URL: https://github.com/apache/jena/pull/1915

   GitHub issue resolved #1913
   
   Pull request Description: See #1913.
   
   Some people have had setup problems setting up Jena in an IDE because of the IDE gets confused by jena-shaded-guava. This friction should go away by using unshaded alternatives. 
   
   Google guava is still in use - as an unshaded dependency in jena-serviceenhancer.
   
   jsonld-java has its own shaded google-guava artifact.
   
   ----
   
    - [x] Commits have been squashed to remove intermediate development commit messages.
    - [x] Key commit messages start with the issue number (GH-xxxx, or if in JIRA, JENA-xxxx)
   
   By submitting this pull request, I acknowledge that I am making a contribution to the Apache Software Foundation under the terms and conditions of the [Contributor's Agreement](https://www.apache.org/licenses/contributor-agreements.html).
   


-- 
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 go to the specific comment.

To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org


[GitHub] [jena] afs commented on pull request #1915: GH-1913: Don't have a shaded Google Guava artifact. Use alternative class where possible.

Posted by "afs (via GitHub)" <gi...@apache.org>.
afs commented on PR #1915:
URL: https://github.com/apache/jena/pull/1915#issuecomment-1601723661

   I've added two commits unrelated to the content here but needed to get github actions to run.
   
   The changes are:
   * `.mvn/` files - currently empty -- placeholders for the future
   * Set the heap size for `jena-jdbc-driver-tdb` run of surefire.
   
   Some github actions machines must be quite small. Using my personal account for running the github actions, `jena-jdbc-driver-tdb` was running out of heap space for tests (specifically `TestTdbMemResultSets`).


-- 
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 go to the specific comment.

To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org


[GitHub] [jena] afs commented on pull request #1915: GH-1913: Don't have a shaded Google Guava artifact. Use alternative class where possible.

Posted by "afs (via GitHub)" <gi...@apache.org>.
afs commented on PR #1915:
URL: https://github.com/apache/jena/pull/1915#issuecomment-1601403806

   This PR also, accidentally (!!), fixes the Jenkins build.
   
   A "clearing up" change in a repo not in the -Pdev build tries to access a constructor that was mad private in favour of a static method for `public` use.
   


-- 
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 go to the specific comment.

To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org


[GitHub] [jena] afs merged pull request #1915: GH-1913: Don't have a shaded Google Guava artifact. Use alternative class where possible.

Posted by "afs (via GitHub)" <gi...@apache.org>.
afs merged PR #1915:
URL: https://github.com/apache/jena/pull/1915


-- 
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 go to the specific comment.

To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org