feat: upload raw log block to s3 #591
Conversation
src/common/utils/utils.ts
Outdated
| return err; | ||
| }); | ||
| if (foundS3Object && !overwrite) { | ||
| return; |
There was a problem hiding this comment.
chỗ này nên log là bị duplicate ra, vì chắc là có vấn đề mới duplicate
There was a problem hiding this comment.
chỗ này nằm ngoài service nên ko có logger à?
vậy thì nên throw rồi để hàm gọi nó tự xử lý
There was a problem hiding this comment.
log như này có hiện ra ko ko biết
There was a problem hiding this comment.
throw ra ngoài thì chỗ Promise.all có thể fail luôn cả job đó anh
| Utils.uploadDataToS3( | ||
| block.height.toString(), | ||
| s3Client, | ||
| `rawlog/${config.chainId}/block/${block.height}`, |
There was a problem hiding this comment.
thêm đc network name sau rawlog ko?
với cả block -> blocks
ví dụ:
rawlog/aura/xstaxy-1/blocks/12341234
There was a problem hiding this comment.
lại phải tạo thêm biến trong config.json nên em không thêm network name đâu
There was a problem hiding this comment.
thêm vào đi cho đẹp, nhỡ có chain nào dở hơi nó để tên trùng
cơ mà bên network.json cũng đang ko có tên chain, kể ra cũng hơi thiếu nhưng chưa thấy dùng ở đâu
|
refer #635 |
No description provided.