Remove unused dep net.thisptr:jackson-jq

[janhoy]

This dependency seems completely unused, could not find explicit usage and saw no change in gradle lock files when removing. If PR tests are green, should be good to go...

I filed this as I was considering #3132 and #3576 for merging...

[epugh]

What clued you in that this might not be needed? I often see dependencies with names that I don't recognize and think "I wonder what that does!"!

[malliaridis]

versionCatalogFormat should remove unused dependencies from version catalog (hence why we also use #keep to keep dependencies we want), and that task is tied on tidy as well. But I don't see unused dependencies being removed. 🤔

What clued you in that this might not be needed?

If you give IntelliJ a few seconds it will gray out the dependency that is not used explicitly. When you then look up the dependency module name in the project, if it does not appear in any gradle lock file, then it is not used at all. That is how I at least find these dependencies that can be removed.

FYI the following two dependencies are also not used:

prometheus-simpleclient = { module = "io.prometheus:simpleclient", version.ref = "prometheus-simpleclient" }
prometheus-simpleclient-httpserver = { module = "io.prometheus:simpleclient_httpserver", version.ref = "prometheus-simpleclient" }

[janhoy]

First I searched for an import and could not find, then I noticed that IntelliJ marked the line grey in Versions TOML file, meaning it is not referenced in any build.gradle file. And not marked as a transitive dep we want to pin either.

May have been in use before removal if Tika perhaps?

[dsmiley]

This was for the prometheus exporter, now removed. I take it you folks haven't had the misery of debugging "jq" expressions ;-)