Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compiler warnings #1446

Closed
tobiasschweizer opened this issue Sep 24, 2019 · 9 comments · Fixed by #1403
Closed

Compiler warnings #1446

tobiasschweizer opened this issue Sep 24, 2019 · 9 comments · Fixed by #1403
Assignees
Labels
bug something isn't working

Comments

@tobiasschweizer
Copy link
Contributor

[info] Compiling 402 Scala sources to .../webapi/target/scala-2.12/classes ...
[warn] .../webapi/src/main/scala/org/knora/webapi/store/triplestore/embedded/JenaTDBActor.scala:441:49: class RAMDirectory in package store is deprecated: see corresponding Javadoc for more information.
[warn]             val indexDirectory: Directory = new RAMDirectory()
[warn]                                                 ^
[warn] .../webapi/src/main/scala/org/knora/webapi/store/triplestore/embedded/JenaTDBActor.scala:452:49: class RAMDirectory in package store is deprecated: see corresponding Javadoc for more information.
[warn]             val indexDirectory: Directory = new RAMDirectory()
[warn]                                                 ^
[warn] two warnings found
@tobiasschweizer tobiasschweizer added the bug something isn't working label Sep 24, 2019
@tobiasschweizer
Copy link
Contributor Author

I got these warnings on #1379

@benjamingeer
Copy link

@subotic subotic added this to the 2019-09 milestone Sep 24, 2019
@subotic
Copy link
Collaborator

subotic commented Sep 24, 2019

Great, was easy to fix :-)

@tobiasschweizer
Copy link
Contributor Author

This is also fixed in #1403: https://github.com/dhlab-basel/Knora/pull/1403/files#diff-d36235129cdb679e49e12100b7f9e8d9

Great!

I think such fixes should rather be done in small and quick PRs since #1403 will be pending for a while. Or was it specifically related to #1403?

@benjamingeer
Copy link

Or was it specifically related to #1403?

It's caused by the upgrade of the RDF4J library in #1403.

@tobiasschweizer
Copy link
Contributor Author

It's caused by the upgrade of the RDF4J library in #1403.

Was the upgrade of the RDF4J library specifically related to #1403? ;-)

@benjamingeer
Copy link

benjamingeer commented Sep 24, 2019

Was the upgrade of the RDF4J library specifically related to #1403? ;-)

Yes, it fixes a stack-overflow error in RDF4J which was triggered by changing a test in #1403.

@tobiasschweizer
Copy link
Contributor Author

Ok, got it. Thanks for the explanation!

@subotic subotic assigned benjamingeer and unassigned subotic Sep 24, 2019
@benjamingeer
Copy link

... and the only reason you're getting the warning is because you copied the RDF4J upgrade from #1403.

@benjamingeer benjamingeer modified the milestones: 2019-09, 2019-10 Sep 25, 2019
@subotic subotic modified the milestones: 2019-10, 2019-11 Oct 27, 2019
@subotic subotic modified the milestones: 2019-11, 2019-12 Nov 27, 2019
@benjamingeer benjamingeer modified the milestones: 2019-12, 2020-01 Dec 16, 2019
@benjamingeer benjamingeer modified the milestones: 2020-01, 2020-02 Jan 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants