-
Notifications
You must be signed in to change notification settings - Fork 93
[WIP] fix up bridge module bugs #65
base: main
Are you sure you want to change the base?
Conversation
4eb3ec7
to
3887e29
Compare
The build failed because you are missing a "DCO line". Signed-off-by: m3ngyang yangm3ng@gmail.com For details, see This is a legal requirement of The Linux Foundation and the Hyperledger consortium. Rebase your commit and force push to update. |
@@ -14,7 +14,7 @@ | |||
|
|||
[KvStorage] | |||
StoragePath = "config/Kv_Shared_tmp" | |||
StorageSize = "1 TB" | |||
StorageSize = "1 GB" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 GByte is very small. The space for the file is not allocated until used, so 1 TByte allows room to grow.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 GB is only for local test, which won't be included in the final commit.
@m3ngyang Can you please update your commit with sign off message. |
Signed-off-by: mengyang02 <mengyang02@baidu.com>
3887e29
to
23fda2a
Compare
@Ram-srini singed off. but this pr is not finished. |
bridge module doesn't work currently. This pr aims to fix the problem. #51