Skip to content

Commit

Permalink
Merge pull request #267 from tomrussello/data-is-not-appended-with-co…
Browse files Browse the repository at this point in the history
…mmitted-streams

fix: data is not appended with committed streams
  • Loading branch information
goccy authored Feb 6, 2024
2 parents 2d5db00 + 1fa309d commit 646ec96
Show file tree
Hide file tree
Showing 2 changed files with 214 additions and 158 deletions.
4 changes: 4 additions & 0 deletions server/storage_handler.go
Original file line number Diff line number Diff line change
Expand Up @@ -517,6 +517,10 @@ func (s *storageWriteServer) appendRows(req *storagepb.AppendRowsRequest, msgDes
s.sendErrorMessage(stream, streamName, err)
return err
}
if err := tx.Commit(); err != nil {
s.sendErrorMessage(stream, streamName, err)
return err
}
} else {
status.rows = append(status.rows, data...)
}
Expand Down
Loading

0 comments on commit 646ec96

Please sign in to comment.