Skip to content

Make sure only one error will be sent to not-started Service#8431

Merged
AlexPeshkoff merged 1 commit intoFirebirdSQL:masterfrom
Noremos:service_error_race
Feb 6, 2025
Merged

Make sure only one error will be sent to not-started Service#8431
AlexPeshkoff merged 1 commit intoFirebirdSQL:masterfrom
Noremos:service_error_race

Conversation

@Noremos
Copy link
Copy Markdown
Contributor

@Noremos Noremos commented Feb 5, 2025

This is a fix for #8430

@hvlad
Copy link
Copy Markdown
Member

hvlad commented Sep 14, 2025

Unfortunately, this broke validation service (at least) ability to print formatted messages.

Current (wrong) output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors

Correct output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors
-       Number of record level warnings : 1

@Noremos
Copy link
Copy Markdown
Contributor Author

Noremos commented Sep 15, 2025

Unfortunately, this broke validation service (at least) ability to print formatted messages.

Current (wrong) output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors

Correct output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors
-       Number of record level warnings : 1

Sad to hear. I'll look into the problem.

@Ev3nt Ev3nt mentioned this pull request Sep 22, 2025
@Noremos
Copy link
Copy Markdown
Contributor Author

Noremos commented Sep 23, 2025

Unfortunately, this broke validation service (at least) ability to print formatted messages.

Current (wrong) output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors

Correct output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors
-       Number of record level warnings : 1

Due to some real-life commitments, I have little free time, so my friend covered for me and made the fix: #8752

@mrotteveel mrotteveel added the rlsnotes60: no Intentionally not added to the Firebird 6.0 release notes. label Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants