Fork me on GitHub
#sql
<
2019-10-12
>
jonpither13:10:29

Hello, does anyone else reflection warnings using next.jdbc and it's dep clojure.java.jdbc?

Reflection warning, clojure/java/data.clj:31:29 - call to method getParameterTypes can't be resolved (target class is unknown).
Reflection warning, clojure/java/data.clj:31:20 - call to static method alength on clojure.lang.RT can't be resolved (argument types: unknown).
Reflection warning, clojure/java/data.clj:35:16 - call to method invoke can't be resolved (target class is unknown).
Reflection warning, clojure/java/data.clj:38:14 - reference to field getName can't be resolved.
Reflection warning, clojure/java/data.clj:39:16 - reference to field getReadMethod can't be resolved.
Reflection warning, clojure/java/data.clj:48:29 - call to method getParameterTypes can't be resolved (target class is unknown).
Reflection warning, clojure/java/data.clj:48:20 - call to static method alength on clojure.lang.RT can't be resolved (argument types: unknown).
Reflection warning, clojure/java/data.clj:51:8 - reference to field getParameterTypes can't be resolved.
Reflection warning, clojure/java/data.clj:55:7 - call to method invoke can't be resolved (target class is unknown).
Reflection warning, clojure/java/data.clj:58:14 - reference to field getName can't be resolved.
Reflection warning, clojure/java/data.clj:59:16 - reference to field getWriteMethod can't be resolved.
Reflection warning, clojure/java/data.clj:65:13 - reference to field getComponentType can't be resolved.
Reflection warning, clojure/java/data.clj:69:5 - call to method addMethod can't be resolved (target class is unknown).
Reflection warning, clojure/java/data.clj:70:5 - call to method addMethod can't be resolved (target class is unknown).
Reflection warning, clojure/java/data.clj:83:21 - call to method getDeclaredMethod can't be resolved (target class is unknown).
Reflection warning, clojure/java/data.clj:83:12 - call to method invoke can't be resolved (target class is unknown).
Reflection warning, clojure/java/data.clj:87:55 - reference to field getCanonicalName can't be resolved.
Reflection warning, clojure/java/data.clj:95:18 - reference to field newInstance can't be resolved.

gklijs14:10:18

I'm used to getting those.. probably have the same ones if I let all those warnings show. With Oracle JDK 11 using defaults you only see the first few. Which JDK are you using?

jonpither14:10:19

openjdk version "10.0.2" 2018-07-17

jonpither14:10:44

would be nice to find a way to not see them, whilst still using the lib, and leaving reflection warnings turned on for else where

seancorfield16:10:39

Those warnings are coming from clojure.java.data, not jdbc.

jonpither16:10:05

Yep. Thought I'd post that here as next jdbc depends on it and you're a core committer, apologies if wrong place though.

seancorfield16:10:18

Do you know if there's a JIRA ticket for those reflection warnings in the Clojure project? I'll go look if you don't know off the top of your head.

seancorfield16:10:33

Doesn't seem so. I'll open one.

jonpither16:10:46

I scanned the GH but not jira

seancorfield16:10:27

Hmm, apparently I can't create a new issue in JIRA for java.data -- I'm not a designated maintainer for it. I'll ping Alex...

seancorfield17:10:14

JDATA-13. I'll fix those if I can and ask Alex to cut a new release.

seancorfield17:10:10

Looks like JDATA-2 was the original issue that fixed some (many) previous warnings.

👍 1
seancorfield18:10:00

@jonpither If you want to depend on the git master of java.data, those reflection warnings should be fixed now.

seancorfield18:10:25

Either Alex will cut 0.1.2 soon or make me the maintainer and I'll cut it.

jonpither18:10:32

Thanks @seancorfield I will give it a whirl

seancorfield18:10:38

Alex just cut 0.1.2 so that'll be up on Maven Central "soon". Then I'll probably tackle some more tickets next weekend and cut 0.1.3.

🎉 1
seancorfield23:10:51

seancorfield/next.jdbc master updated to use org.clojure/java.data "0.1.2" so no more reflection warnings 🙂