Skip to content

LUTECE-2224 : Allow client calls to pass StringBuffer to SearchIndexe… #155

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

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

DiogoSobralCapgemini
Copy link

The goal is to have the logs on blog indexation and not impact the other clients :

BEFORE : image

AFTER :
image

@AlexandreClose
Copy link
Contributor

Some comment about this pull request.

  1. May you refactor the core PageIndexer to write the logs inside the new indexDocuments( StringBuffer ) ? Actually, the logs are written in the IndexationService.write( Document ); but if we want to have a new uniformized system, each indexer should write their own logs inside indexDocuments.

  2. Be careful about the code indent: for instance, the bracket have to be on a empty line like this :
    void myMethod( Param param )
    {
    //Content
    }

You can automatize this process by running
mvn formatter:format
on your lutece project; core included.

Alexandre

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants