NASA-PDS / harvest

Standalone Harvest client application providing the functionality for capturing and indexing product metadata into the PDS Registry system (https://github.com/nasa-pds/registry).
https://nasa-pds.github.io/registry
Other
4 stars 3 forks source link

Double check how the schema is updated, log levels fine tune #196

Closed tloubrieu-jpl closed 1 month ago

tloubrieu-jpl commented 1 month ago

💡 Description

When loading data with harvest, the logs are filled with messages like:

[INFO] This LDD already loaded.
[INFO] Updating Elasticsearch schema.
[INFO] Updated 32 fields

I am wondering if we could:

⚔️ Parent Epic / Related Tickets

No response

al-niessner commented 1 month ago

@tloubrieu-jpl

Yeah, none of it is very clear as written and way out of date. Not sure it ever can be clear or timely because elastic/open changes names often as do the concepts that support it. All three give you informational status about the harvest process. They really are not for debugging. Well, I certainly did not add them for debugging purposes nor use them for it either.

Given the new information, let me know what you want done.

tloubrieu-jpl commented 1 month ago

Hi @al-niessner, maybe we can be more explicit:

If none of these messages are for debugging, I don't believe the first 2 are useful for the user either, so we could remove them altogether. How does that sound ?