hyperledger / fabric-x-committer

92%

main: 91%

LAST BUILD ON BRANCH fix/coordinator-waiting-status-count

branch: SELECT

CHANGE BRANCH

x

• No branch selected
• additional-db-test
• apt-ci-fix
• backoff-v5
• bccsp-race
• bench-report-metrics
• bft-delivery
• bft-metrics
• bob-rules
• buffer-race
• build-unify
• channel-race
• ci-gotestsum
• cli-refactor
• cli-refs
• client-test-race
• client-tls-bug
• commit-config-immediately
• config-docs-yaml
• connection-test-dep
• coordinator-docs
• coordinator-test-no-db
• coveralls
• cryptogen-recreate-support
• dedup-templ
• default-test-node-startup-setup
• default-viper
• deliver-check-block-order
• deliver-seek-tls-cert
• deliver-seek-tls-cert-fix
• dependabot/go_modules/go_modules-9c5197dcb8
• dependabot/go_modules/go_modules-d35c03d4db
• dependabot/go_modules/go_modules-dc1255629f
• dependabot/go_modules/go_modules-eee72e131a
• dependencygraph-race
• deployment-strategy
• doc-loadgen-artifacts
• dynamic-root-ca
• dynamic-tls
• enable-pprof
• expose-ledger-api
• external-server-config
• extract-prometheus-startup-and-grpc-server-start
• fabric-lib-go-v1.1.3
• fabric-logging
• fix-automatic-env
• fix-bft-detection
• fix-connection-reset
• fix-container-image
• fix-container-test-macos
• fix-db-test
• fix-falky-svmupdate-test
• fix-flaky
• fix-flaky-conn-metrics
• fix-flaky-loadgen-coord
• fix-master-tserver-rf
• fix-metrics
• fix-mkdoc-url
• fix-port-container-test
• fix-release-path
• fix-sign
• fix-test-node-with-insecure
• fix-test-wait
• fix-waiting-tx-count
• fix/coordinator-waiting-status-count
• fix/todo-001
• fix/todo-002
• fix/todo-005
• fix/todo-017
• flaky-test-orderer
• getpeersmspdirs-no-hardcode
• go-fix
• grpc-logger-fix
• grpcservice-refactor
• healthz-check
• http-endpoint
• ledger-service-wait-behaviour
• lifecycle-policy
• limit-block-apis
• limit-inputs-block-apis
• load-config-block-certs
• loadgen-manual-msp
• loadgen-remove-ctx
• main
• make-ci-local
• make-fixes
• makefile-lints
• makefile-test-flags
• metrics-bob-rules
• metrics-tls
• metrics-url-race
• migrate-to-postgres-official-image
• minimize-logs
• mkdocs
• mock-orderer-direct-config
• mock-orderer-dynamic-tls
• mock-orderer-features
• mock-orderer-stream-holder
• mock-verifier-race
• move-retry-package
• move-testcrypto-to-common
• namespace-policy-docs
• notifier-metrics
• optimize-map-block
• orderer-config-race
• pgx5
• postgres-renaming
• prom-refactor
• qs-pen-test-fix
• race-detection
• rate-limit-sidecar
• refactor-channel
• refactor-cmd
• refactor-test
• release-1.0
• remove-common-root-cas
• remove-file
• remove-ifrit-gomega
• rename
• rename-materials
• rename-util-files
• review-ai-prompt
• rm-gpl-tools
• rm-reployment
• rm-unused
• root-ca-workaround-fix
• service-manager
• sidecar-block-store-code-fix
• sidecar-diagram-mod
• sidecar-diagrm
• sidecar-limit-active-txids-notifier
• sidecar-race
• sidecar-remove-chan-id
• sidecar-unified
• split-tablets
• start-server-readiness-context
• sustain-config
• sync-block
• template-tls-per-endpoint
• test-container-port-binding
• test-coverage-connection
• test-coverage-improve
• test-coverage-latency
• test-coverage-tests
• test-limiter-max-wait
• test-minor-fixes
• test-query-status
• test-tls-flatten
• testcrypto-race
• testcrypto-refactor
• tls-docs-update
• unused-code
• update-common
• update-common-dep
• update-dep
• update-deps
• update-to-latest-common
• update-to-yugabytedb-latest-lts
• upgrade-pg
• vc-docs
• vc-race
• vc-resource-leak
• verifier-test-remove-gomega
• verifier-timeout
• worktree-notification-docs

Build # 26457782255

Build Type

Pull #613

github

Committed by cendhu

Commit Message

fix coordinator waiting status count

Track VC-produced transaction statuses separately from the status queue
length so NumberOfWaitingTransactionsForStatus does not count statuses
that are already ready to be sent back to the sidecar.

Use a txStatusQueue to count statuses before enqueueing and roll the
count back if a blocked enqueue is canceled. This avoids a race where a
fast reader could decrement the count before the writer increments it.

Simplify queue handling by removing impossible nil/closed-channel cases
and keep tests focused on count accounting and context cancellation.

Signed-off-by: Senthilnathan <cendhu@gmail.com>

Pull Request Pull Request #613: [Coordinator] fix waiting status count

Coverage Stats

43 of 44 new or added lines in 3 files covered. (97.73%)

10272 of 11190 relevant lines covered (91.8%)

40924.41 hits per line