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

Extend the rate controller repertoire #42

Closed
wants to merge 5 commits into from
Closed

Extend the rate controller repertoire #42

wants to merge 5 commits into from

Conversation

aklenik
Copy link
Contributor

@aklenik aklenik commented Apr 25, 2018

The aim of this PR is to pave the road for advanced workload modelling. Contributions include:

  • Rate control related:

    • Added end method to the rate controller proxy to signal the end of a round (in a backward compatible/optional way).
    • Modified round execution workflow to call this end method.
    • Added a composite rate controller implementation.
    • Added a zero rate controller implementation.
    • Extended the rate controller documentation.
    • Added a sample configuration to demonstrate the usage of the new controllers.
  • Modified end script of config files (for Fabric) to also remove the chaincode images.

  • Fixed linting errors for Fabric subsystem (Issue Fulfill linting requirements #48 ).

Signed-off-by: Attila Klenik <a.klenik@gmail.com>
…k)fixes

Signed-off-by: Attila Klenik <a.klenik@gmail.com>
Signed-off-by: Attila Klenik <a.klenik@gmail.com>
Signed-off-by: Attila Klenik <a.klenik@gmail.com>
Signed-off-by: Attila Klenik <a.klenik@gmail.com>
@haojun
Copy link
Contributor

haojun commented May 4, 2018

Hi @aklenik, could you please split it into two different PRs, since those commits are totally for different purpose. Thanks.

@aklenik
Copy link
Contributor Author

aklenik commented May 4, 2018

Will do that (possibly by opening an other PR, this got complicated)

@aklenik aklenik closed this May 7, 2018
faustovanin pushed a commit to faustovanin/caliper that referenced this pull request Oct 27, 2023
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

Successfully merging this pull request may close these issues.

2 participants