-
Notifications
You must be signed in to change notification settings - Fork 236
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
Dependency analysis in gitlab #2144
base: integration
Are you sure you want to change the base?
Conversation
8d6929c
to
9fec5b7
Compare
1f7ffe1
to
c215908
Compare
761736e
to
367009b
Compare
c3ba32d
to
83bbbc6
Compare
1a2278a
to
3c84d00
Compare
f70b06f
to
4545a0f
Compare
4545a0f
to
56a8faa
Compare
200ea83
to
8f83b84
Compare
82a9abe
to
a59bc19
Compare
9cbc96d
to
0321a60
Compare
eedd9fb
to
7ce39ad
Compare
1c0a807
to
a68b7ac
Compare
c3f4042
to
8d7d769
Compare
14ade5e
to
aac32b4
Compare
c7e92b5
to
caa48a8
Compare
4fd4a25
to
b57034f
Compare
074caee
to
e8d4079
Compare
<groupId>io.astefanutti.metrics.cdi</groupId> | ||
<artifactId>metrics-cdi</artifactId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't verified yet, but I'm pretty sure this change is the reason that the web service deployment is failing in the test runner. That lib is needed for CDI injection of the dropwizard @Metric
-annotated fields.
From server.log...
org.jboss.weld.exceptions.DeploymentException: WELD-001408: Unsatisfied dependencies for type Counter with qualifiers @Default
at injection point [BackedAnnotatedField] @Inject @Metric private datawave.webservice.operations.remote.RemoteAccumuloService.failureCounter
at datawave.webservice.operations.remote.RemoteAccumuloService.failureCounter(RemoteAccumuloService.java:0)
Sadly, Wildfly doesn't give anything useful like a ClassNotFoundException in this case :-(
1553678
to
a33b4c9
Compare
a33b4c9
to
ea0b412
Compare
Dependency analysis runs in gitlab
updated structure to remove most "unused declared" and "used undeclared" dependencies.
A few still remain that are used that the tool falsely identifies as unused. Notes on those below. I tried using a few different tactics but I wasn't quite able to untangle the dependencies.
locally, run
mvn -DignoreNonCompile dependency:analyze
to see results.
NOTES
datawave-ws-atom
needs abdera, axiom-impl, axiom-api, hadoop-client-runtime
datawave-ingest-core
needs accumulo-server-base, log4j-core, hadoop-client-runtime, log4japi, log4j-slf4j-impl
datawave-ingest-csv
needs datawave-ingest-configuration, accumulo-core is not test only
datawave-ingest-json
needs datawave-ingest-configuration, hadoop-mapreducew-client-core/-common
datawave-query-core
needs json-simple, jackson-module-jaxb-annotations, lucene-analyzers-common, commons-logging, datawave-ingest-configuration, hadoop-mapreduce-client-core
datawave-ingest-wikipedia
needs datawave-ingest-configuration, type-utils not just test only
assemble-datawave
datawave-ops-tools-index-validation
needs zookeeper
OLD NOTES
datawave-core:
log4j-slf4j-impl
causesorg.junit.ComparisonFailure: expected:<[Hash]UID> but was:<[Snowflake]UID>
error - UID uses a logger.datawave-ws-atom:
abdera-parser
,hadoop-client-runtime
,axiom-api
,axiom-impl
are all requireddatawave-ingest-core:
accumulo-server-base
,log4j-core
,hadoop-client-runtime
,log4j-api
,log4j-slf4j-impl
are all requireddatawave-ingest-csv:
hadoop-mapreduce-client-common
,datawave-ingest-configuration
are usedaccumulo-core
not just scoped for testsdatawave-ingest-json:
hadoop-mapreduce-client-core
,hadoop-mapreduce-client-common
,datawave-ingest-configuration
are all requireddatawave-query-core:
-
hadoop-mapreduce-client-core
,json-simple
are requireddatawave-ws-model:
slf4j-api
is requiredauthorization-api
is not just scoped for testsdatawave-ingest-wikipedia:
-
datawave-ingest-configuration
is requiredtype-utils
is not just scoped for testsdatawave-metrics-core:
datawave-query-core
is requireddatawave-ops-tools-config-compare:
hadoop-client-runtime
is requireddatawave-ops-tools-index-validation:
zookeeper
,guava
are required