Skip to content

Commit feda209

Browse files
authored
Merge pull request #1421 from 0chain/fix/conn-size
Fix update connection size
2 parents 8aaf842 + a89b435 commit feda209

File tree

3 files changed

+6
-5
lines changed

3 files changed

+6
-5
lines changed

code/go/0chain.net/blobbercore/allocation/connection.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ func UpdateConnectionObjSize(connectionID string, addSize int64) {
166166
connectionObj.UpdatedAt = time.Now()
167167
}
168168

169-
func SaveFileChange(connectionID, pathHash, fileName string, cmd FileCommand, isFinal bool, contentSize int64, offset, dataWritten int64) (bool, error) {
169+
func SaveFileChange(connectionID, pathHash, fileName string, cmd FileCommand, isFinal bool, contentSize, offset, dataWritten, addSize int64) (bool, error) {
170170
connectionObjMutex.RLock()
171171
connectionObj := connectionProcessor[connectionID]
172172
connectionObjMutex.RUnlock()
@@ -210,6 +210,9 @@ func SaveFileChange(connectionID, pathHash, fileName string, cmd FileCommand, is
210210
Offset: offset,
211211
DataBytes: dataWritten,
212212
}, contentSize)
213+
if addSize != 0 {
214+
UpdateConnectionObjSize(connectionID, addSize)
215+
}
213216
} else {
214217
change.seqPQ.Push(seqpriorityqueue.UploadData{
215218
Offset: offset,

code/go/0chain.net/blobbercore/handler/file_command_update.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,6 @@ func (cmd *UpdateFileCommand) ProcessContent(allocationObj *allocation.Allocatio
147147
if fileOutputData.ContentSize != cmd.fileChanger.Size {
148148
return result, common.NewError("upload_error", fmt.Sprintf("File size mismatch. Expected: %d, Actual: %d", cmd.fileChanger.Size, fileOutputData.ContentSize))
149149
}
150-
allocation.UpdateConnectionObjSize(connID, cmd.fileChanger.Size-cmd.existingFileRef.Size)
151150
}
152151

153152
if cmd.thumbFile != nil {
@@ -157,7 +156,7 @@ func (cmd *UpdateFileCommand) ProcessContent(allocationObj *allocation.Allocatio
157156
}
158157
}
159158

160-
saveChange, err := allocation.SaveFileChange(connID, cmd.fileChanger.PathHash, cmd.fileChanger.Filename, cmd, cmd.fileChanger.IsFinal, cmd.fileChanger.Size, cmd.fileChanger.UploadOffset, fileOutputData.Size)
159+
saveChange, err := allocation.SaveFileChange(connID, cmd.fileChanger.PathHash, cmd.fileChanger.Filename, cmd, cmd.fileChanger.IsFinal, cmd.fileChanger.Size, cmd.fileChanger.UploadOffset, fileOutputData.Size, cmd.fileChanger.Size-cmd.existingFileRef.Size)
161160
if err != nil {
162161
return result, err
163162
}

code/go/0chain.net/blobbercore/handler/file_command_upload.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,6 @@ func (cmd *UploadFileCommand) ProcessContent(allocationObj *allocation.Allocatio
164164
if fileOutputData.ContentSize != cmd.fileChanger.Size {
165165
return result, common.NewError("upload_error", fmt.Sprintf("File size mismatch. Expected: %d, Actual: %d", cmd.fileChanger.Size, fileOutputData.ContentSize))
166166
}
167-
allocation.UpdateConnectionObjSize(connectionID, cmd.fileChanger.Size)
168167
}
169168

170169
if cmd.thumbFile != nil {
@@ -175,7 +174,7 @@ func (cmd *UploadFileCommand) ProcessContent(allocationObj *allocation.Allocatio
175174
}
176175
}
177176

178-
saveChange, err := allocation.SaveFileChange(connectionID, cmd.fileChanger.PathHash, cmd.fileChanger.Filename, cmd, cmd.fileChanger.IsFinal, cmd.fileChanger.Size, cmd.fileChanger.UploadOffset, fileOutputData.Size)
177+
saveChange, err := allocation.SaveFileChange(connectionID, cmd.fileChanger.PathHash, cmd.fileChanger.Filename, cmd, cmd.fileChanger.IsFinal, cmd.fileChanger.Size, cmd.fileChanger.UploadOffset, fileOutputData.Size, cmd.fileChanger.Size)
179178
if err != nil {
180179
logging.Logger.Error("UploadFileCommand.ProcessContent", zap.Error(err))
181180
return result, err

0 commit comments

Comments
 (0)