Skip to content

Search results debugging #4194

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

Merged
merged 2 commits into from
Mar 3, 2023
Merged

Search results debugging #4194

merged 2 commits into from
Mar 3, 2023

Conversation

vladak
Copy link
Member

@vladak vladak commented Mar 3, 2023

Inspired by #4180 (comment), this change will log the documents found by a search at the FINEST level. The document ID and path will be logged so that one can see whether there are multiple documents with the same path and different ID.

Here's example of the output (for the second page of search results, hence the 25-37) range:

03-Mar-2023 17:48:01.868 FINEST [http-nio-8080-exec-124] org.opengrok.indexer.search.Results.createMap directory hash contents for search hits (25,37):
03-Mar-2023 17:48:01.875 FINEST [http-nio-8080-exec-124] org.opengrok.indexer.search.Results.createMap 5,631: /lucene/lucene/core/src/java/org/apache/lucene/index/PointValues.java
03-Mar-2023 17:48:01.876 FINEST [http-nio-8080-exec-124] org.opengrok.indexer.search.Results.createMap 6,403: /lucene/lucene/analysis/common/src/java/org/apache/lucene/analysis/email/UAX29URLEmailTokenizerImpl.jflex

It does not display the actual search query, that would require some sort of representation of the SearchHelper instance. Also, this is meant for very specific debugging, so it's not actually needed.

@vladak vladak added the webapp web application label Mar 3, 2023
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Mar 3, 2023
@vladak vladak merged commit 17f3247 into oracle:master Mar 3, 2023
@vladak vladak deleted the search_results_debug branch March 3, 2023 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement. webapp web application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant