codice / ddf

DDF Distributed Data Framework - an open source, modular integration framework.
http://ddf.codice.org
Other
132 stars 181 forks source link

[2.26.x] DDF-6704 improve query logging #6705

Closed glenhein closed 1 year ago

glenhein commented 2 years ago

What does this PR do?

Add detailed logging for the pre and post query plugins, and for the solr query code.

Note: The original intent was to include the trace-id in the solr logging, however there is a problem in the code with the thread context getting passed through to the solr code.

Who is reviewing it?

@jlcsmith @derekwilhelm

Select relevant component teams:

@codice/io @codice/solr

Ask 2 committers to review/merge the PR and tag them here.

@brendan-hofmann @clockard

How should this be tested?

Enable trace logging:

log:set TRACE ddf.catalog.impl.operations log:set TRACE ddf.catalog.source.solr

Perform a query in the UI. Check the DDF log file for the extra logging.

Any background context you want to provide?

What are the relevant tickets?

Fixes: #6704

Screenshots

Checklist:

Notes on Review Process

Please see Notes on Review Process for further guidance on requirements for merging and abbreviated reviews.

Review Comment Legend:

jlcsmith commented 2 years ago

build now

cxddfbot commented 2 years ago

Internal build has been started, your results will be available at build completion.

cxddfbot commented 2 years ago

Build SUCCESS See the job results in legacy Jenkins UI or in Blue Ocean UI.

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs within 7 days. Thank you for your contributions.