Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

support GetStateMultipleKeys GetPrivateDataMultipleKeys #5116

Closed
pfi79 opened this issue Jan 23, 2025 · 2 comments
Closed

support GetStateMultipleKeys GetPrivateDataMultipleKeys #5116

pfi79 opened this issue Jan 23, 2025 · 2 comments

Comments

@pfi79
Copy link
Contributor

pfi79 commented Jan 23, 2025

I have a desire to implement functionality so that GetStateMultipleKeys, GetPrivateDataMultipleKeys can be called from the chaincodes.

As it seems to me, the task is trivial and does not require creating a separate rfcs.

I will need to make changes to a few repositories: protos, chaincode and fabric.

  1. peer will pass a flag if it supports these operations
  2. if chaincode finds out that peer does not support these operations, it will send GetState in a loop. In other words, a new chaincode can be placed on any peer and nothing will break.

My only problem is that I won't be able to write code in java and node. I can read the code, but I can't write it. If someone will support this functionality for java and node, it will be great.

@pfi79
Copy link
Contributor Author

pfi79 commented Jan 23, 2025

@denyeart

@denyeart
Copy link
Contributor

denyeart commented Feb 5, 2025

@pfi79 I agree the change is small enough that you could do it without an RFC. I think you can proceed with PRs. I believe the ledger APIs are already in place.

We typically lead with changes in fabric-chaincode-go, and then the other chaincode languages can catch up if people are interested. So my suggestion is to complete the fabric-chaincode-go changes, and then we can open issues in fabric-chaincode-node and fabric-chaincode-java to see if anybody wants to complete them (same holds true for the recent write batching).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants