Skip to content

Conversation

@BRupireddy2
Copy link
Contributor

This commit enables pgactive per-db and apply workers to report their last error into shared memory. Users can see the last error info via pgactive_get_workers_info SQL function.

One error for each per-db and apply worker is added for now. This can be extended to other errors by adding a specific pgactive error code and emitting error via the ereport_pgactive() in place of ereport()/elog().

This last error info improves observability of the pgactive setups.

===========================================================================================
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@BRupireddy2 BRupireddy2 self-assigned this Sep 7, 2024
@BRupireddy2 BRupireddy2 force-pushed the add_pgactive_worker_error_reporting_facility_WIP branch 2 times, most recently from ea50ef6 to d78bdd9 Compare September 10, 2024 14:15
@BRupireddy2 BRupireddy2 force-pushed the add_pgactive_worker_error_reporting_facility_WIP branch from d78bdd9 to d01915b Compare September 10, 2024 17:33
Copy link
Contributor

@sharmay sharmay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Visually looks good

This commit enables pgactive per-db and apply workers to report
their last error into shared memory. Users can see the last error
info via pgactive_get_workers_info SQL function.

One error for each per-db and apply worker is added for now. This
can be extended to other errors by adding a specific pgactive
error code and emitting error via the ereport_pgactive() in place
of ereport()/elog().

This last error info improves observability of the pgactive
setups in production.
@BRupireddy2 BRupireddy2 force-pushed the add_pgactive_worker_error_reporting_facility_WIP branch from d01915b to 7648977 Compare September 13, 2024 08:43
@BRupireddy2 BRupireddy2 merged commit 9ddcc06 into main Sep 13, 2024
@BRupireddy2 BRupireddy2 deleted the add_pgactive_worker_error_reporting_facility_WIP branch September 13, 2024 13:20
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.

3 participants