Skip to content

Commit b12ee59

Browse files
author
Stephen Hoover
committed
BUG Fix log string formatting
Without this "%s", we see an error message for every fragment of multipart uploads.
1 parent a2b7d5a commit b12ee59

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@ This project adheres to [Semantic Versioning](http://semver.org/).
66
### Added
77
- Added a script for integration tests (smoke tests).
88

9+
### Fixed
10+
- Added missing string formatting to a log emit in file multipart upload (#217)
11+
912
### Changed
1013
- Updated CivisML 2.0 notebook (#214)
1114
- Reworded output of `civis notebooks new` CLI command (#215)

civis/io/_files.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ def _upload_part_base(item, file_path, part_size, file_size):
148148
msg = _get_aws_error_message(part_response)
149149
raise HTTPError(msg, response=part_response)
150150

151-
log.debug('Completed upload of file part', part_num)
151+
log.debug('Completed upload of file part %s', part_num)
152152

153153
# upload each part
154154
try:

0 commit comments

Comments
 (0)