Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[TxLogs] Move TxLogs commit (append to file) in a Separate Thread #101

Open
TanmoySG opened this issue Apr 21, 2023 · 0 comments · Fixed by #103
Open

[TxLogs] Move TxLogs commit (append to file) in a Separate Thread #101

TanmoySG opened this issue Apr 21, 2023 · 0 comments · Fixed by #103
Assignees
@TanmoySG

Comments

@TanmoySG
Copy link
Owner

Move to a separate thread using go routines
@TanmoySG TanmoySG self-assigned this Apr 21, 2023
TanmoySG added a commit that referenced this issue Apr 22, 2023
@TanmoySG
goroutine for handleTransactions - #101
70ad369
@TanmoySG TanmoySG mentioned this issue Apr 22, 2023
Merged
TanmoySG added a commit that referenced this issue Apr 22, 2023
@TanmoySG
TxLog to separate thread using goroutine [#101]
f69590e 
Merge pull request #103 from TanmoySG/txlog-routine

Closes #101 

- Moved txlog method `handleTransactions` to separate thread with goroutine

Note:
- Might require additional panic handling for goroutine, currently not being handled by the default panic handler in `server/middleware/recovery`, though haven't encountered any panics for current code/tests
TanmoySG added a commit that referenced this issue Apr 22, 2023
@TanmoySG
txlog handle internal error #101
6bc59a6
@TanmoySG TanmoySG reopened this Apr 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TanmoySG TanmoySG

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

TxLog to Goroutine TanmoySG/wunderDB
1 participant
@TanmoySG