Skip to content

Conversation

@SValanukonda
Copy link
Contributor

@SValanukonda SValanukonda commented Mar 13, 2025

This PR provides a simpler fix for issue #26 by using an echo channel to route OS signals without extra goroutines.
Kindly review

@SValanukonda
Copy link
Contributor Author

hi @arpitbbhayani ,
Please check if this approach is helpful

@arpitbbhayani arpitbbhayani merged commit 84d4f58 into dicedb:master Mar 29, 2025
@arpitbbhayani
Copy link
Contributor

Merged. Thanks for the patch!

@SValanukonda
Copy link
Contributor Author

Hi @arpitbbhayani
I appreciate the opportunity to contribute and thank you for merging my PR. I’d love to stay involved and continue contributing — please feel free to point me to any tasks, open issues, or areas where I could be helpful!

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