Skip to content

perf(storagenode): remove backup from append request#412

Merged
ijsong merged 1 commit intomainfrom
remove_backup_from_append_request
Jun 1, 2023
Merged

perf(storagenode): remove backup from append request#412
ijsong merged 1 commit intomainfrom
remove_backup_from_append_request

Conversation

@ijsong
Copy link
Copy Markdown
Member

@ijsong ijsong commented Apr 11, 2023

What this PR does

This PR removes the Backups fields from AppendRequest. Thus, it makes unmarshaling AppendRequest can
be improved.

@ijsong ijsong self-assigned this Apr 11, 2023
@ijsong ijsong force-pushed the remove_backup_from_append_request branch from b1e0982 to 4e18b9d Compare April 13, 2023 13:23
@ijsong ijsong force-pushed the remove_backup_from_append_request branch from 4e18b9d to 063890f Compare April 13, 2023 13:43
@ijsong ijsong marked this pull request as ready for review May 25, 2023 06:08
This PR removes the Backups fields from AppendRequest. Thus, it makes unmarshaling AppendRequest can
be improved.
@ijsong ijsong force-pushed the check_log_level branch from e1fd68f to 39cdbae Compare June 1, 2023 03:31
@ijsong ijsong force-pushed the remove_backup_from_append_request branch from 0fa973d to f75ef55 Compare June 1, 2023 03:31
@ijsong
Copy link
Copy Markdown
Member Author

ijsong commented Jun 1, 2023

@ijsong started a stack merge that includes this pull request via Graphite.

Base automatically changed from check_log_level to main June 1, 2023 05:26
@ijsong ijsong merged commit 0abcb1a into main Jun 1, 2023
@ijsong ijsong deleted the remove_backup_from_append_request branch June 1, 2023 05:27
@ijsong
Copy link
Copy Markdown
Member Author

ijsong commented Jun 1, 2023

@ijsong merged this pull request with Graphite.

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