Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Purge data in explorer based on a parameter #341

Closed
wants to merge 1 commit into from

Conversation

abhakish
Copy link

@abhakish abhakish commented Jan 30, 2023

Signed-off-by: Abhay Kishore abhay.kishore@walmart.com

What this PR does / why we need it:

It will purge the old data present in explorer db based on the parameter blockCount.blockCount property can be defined in config.json.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?


Additional documentation, usage docs, etc.:


Signed-off-by: Abhay Kishore <abhay.kishore@walmart.com>
@abhakish abhakish requested a review from a team as a code owner January 30, 2023 09:40
@arigelas arigelas requested review from arigelas and removed request for arigelas January 30, 2023 11:11
@@ -2,7 +2,8 @@
"network-configs": {
"test-network": {
"name": "Test Network",
"profile": "./connection-profile/test-network.json"
"profile": "./connection-profile/test-network.json",
"blockCount": "5"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick]: update this to numerical value.

@@ -86,6 +88,8 @@ export class SyncPlatform {
this.network_name = args[1];
}

this.blockCount = network_configs[this.network_id].blockCount;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

blockCount needs to be defined in config.json in (app/platform/fabric/config.json)

* @returns
* @memberof CRUDService
*/
async deleteBlock(network_name, channel_genesis_hash, blockCount) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ongoing we can add types also in the function args

@adityajoshi12
Copy link
Contributor

Duplicate of #362

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants