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

[Postgres] register_file needs the CorrealtionID when updating the file event log #1229

Open
3 tasks
jbygdell opened this issue Dec 30, 2024 · 0 comments
Open
3 tasks
Labels
bug Something isn't working Refactoring Work: Refactoring

Comments

@jbygdell
Copy link
Collaborator

Please describe the feature

As a developer, I want no hidden bugs so that I don't have to spend time fixing thins that should not need fixing.

When a file i registered in the DB an event is automatically added to the log table, this event doesn't have a CorrelationID, which it should have. This then lead to issues downstream when we try to extract the Correlation ID from the logs table since the QueryRow function always will return the first entry. (not always the case if the first row is NULL)

Acceptance criteria

  • register_file inserts the Correlation ID correctly, bear in mind that this will be different if the function is called from ingest or from the s3inbox.
    OR
  • register_file does not update the file event log in this case so we can do that from the outside where the Correlation ID is available.
  • Tests verifying the changes are added

Additional context

Estimation of size

small

Estimation of priority

low

@jbygdell jbygdell added bug Something isn't working Refactoring Work: Refactoring labels Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Refactoring Work: Refactoring
Projects
None yet
Development

No branches or pull requests

1 participant