Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Reindex causes too many log messages for file metadata #18752

Closed
john-thomas-dotcms opened this issue Jun 23, 2020 · 2 comments
Closed

Reindex causes too many log messages for file metadata #18752

john-thomas-dotcms opened this issue Jun 23, 2020 · 2 comments
Labels
Merged QA : Approved QA : Passed Internal Release : 5.3.3 Type : Defect

Comments

@john-thomas-dotcms
Copy link
Contributor

Describe the bug

When a site is reindexed, a log message is written for every single file which we read metadata from a .json file.

Example log message:

[23/06/20 15:22:59:811 GMT] INFO tika.TikaUtils: Metadata read from: /data/shared/assets/f/a/fa366899-78f2-4c53-bb5b-67b00eb3326a/diagram1-metadata.json

Steps to reproduce the behavior:

  1. Reindex the starter site.
  2. View the dotcms.log file.

Expected behavior

No log messages should be written when file metadata is read during a reindex.

Screenshots

image

Desktop (please complete the following information):

  • OS: Win 10 Pro
  • Browser: Chrome
  • Version: dotCMS 5.3.3 (auth)

Acceptance Criteria

  1. A clean reindex should only show messages summarizing the reindex process.
  2. Work in all the supported browsers (don't forget IE11 and iPad)
  3. Multilanguage
  4. Unit test
@erickgonzalez erickgonzalez added this to the Bug Sprint milestone Jun 23, 2020
@jdotcms
Copy link
Contributor

jdotcms commented Jun 23, 2020

PR

@bryanboza
Copy link
Member

Fixed, tested on release-5.3.3 // Postgres // FF

@fmontes fmontes closed this as completed Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Merged QA : Approved QA : Passed Internal Release : 5.3.3 Type : Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@fmontes @bryanboza @erickgonzalez @nollymar @john-thomas-dotcms @jdotcms