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.

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

chore: Improve deceptive Archive message log #3164

Open
stubbsta opened this issue Oct 31, 2024 · 2 comments
Open

chore: Improve deceptive Archive message log #3164

stubbsta opened this issue Oct 31, 2024 · 2 comments

Comments

@stubbsta
Copy link

stubbsta commented Oct 31, 2024

Background

There is no log to clarify that there was an attempt to archive a message, but it failed.

Details

The current logs when a store node archives a single message are:

put PostgresDriver                         tid=1 file=postgres_driver.nim:305 messageHash=24948...
message archived                           topics="waku archive" tid=1 file=archive.nim:138 msg_hash...

It is assumed that the log "message archived" verifies that archiving was successful, but if the message is already in the DB, then no message is archived even though the "message archived" log is still present.
The log needs to make it clear whether archiving worked or not.

When running a store node with store sync enabled, it is more clear when there is message duplication, because the "archive message" log is followed by these logs:

inserting message into waku sync storage   topics="waku sync" ...
failed to insert message                   topics="waku sync" tid=1 file ...

Acceptance criteria

Modify the log or add an additional log to identify if there was an error.

@stubbsta
Copy link
Author

@AlbertoSoutullo

@stubbsta
Copy link
Author

stubbsta commented Nov 8, 2024

@SionoiS and @chaitanyaprem this also relates to my question about getting the time that a message is inserted into the DB.
Perhaps the "message archived" log could include an "inserted time" or something like that.

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

No branches or pull requests

1 participant