Skip to content

bind_hash_mapping sampling logging changes w.r.t. longer corrid #414

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

Closed
wants to merge 8 commits into from

Conversation

vineravi
Copy link
Collaborator

No description provided.

rasamala83 and others added 8 commits March 24, 2025 15:47
…ate during worker recovery" (#410)

* Revert "fixin functional test comparision"

This reverts commit 44bebff.

* Revert "removed unwanted changes related to induced delays during execution"

This reverts commit c00b147.

* Revert "fixing race condition related to worker recovery issue"

This reverts commit 02c1500.

* Revert "move process reaping login go from C"

This reverts commit 8db5c70.

* Revert "remove restart worker logic as it will taken care worker monitor"

This reverts commit 957b0de.

* Revert "added missing transtion from finish state to scheduled state"

This reverts commit eeb2943.

* Revert "update test code based on state transition map"

This reverts commit cd92e68.

* Revert "instead of flow based control added valid transition map to control state updates"

This reverts commit 24e2039.

* Revert "remove unwanted log message"

This reverts commit 363564d.

* Revert "first version of changes to avoid accidental worker state update during worker recovery"

This reverts commit 8739920.
* changes for adding atomic checks while sending and relading data during worker recovery

* fixing the workflow issues
@vineravi
Copy link
Collaborator Author

i will close this PR as the commits are confusing.
changes raised on other PR #422

@vineravi vineravi closed this Mar 24, 2025
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.

2 participants