Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

Added benchmarks for Rapidoid using the high-level API.#2256

Merged
knewmanTE merged 6 commits intoTechEmpower:round-14from
nmihajlovski:round-14
Sep 19, 2016
Merged

Added benchmarks for Rapidoid using the high-level API.#2256
knewmanTE merged 6 commits intoTechEmpower:round-14from
nmihajlovski:round-14

Conversation

@nmihajlovski
Copy link
Copy Markdown
Contributor

No description provided.

@NateBrady23
Copy link
Copy Markdown
Member

@nmihajlovski I appreciate your efforts here, especially with transparency. Moving forward, we would love to be able to show people snippets of what realistic implementations of tests would look like.

Is there any way we can combine this within the same rapidoid folder? We don't want to have separate framework directories for "low" and "high-level" implementations, but if you could keep it in rapidoid and then name the tests accordingly that would work great.

Would not be opposed to putting some snippets/examples in the README as well.

@nmihajlovski
Copy link
Copy Markdown
Contributor Author

Thanks for your quick reply.
I combined the low-level and high-level implementations, and added some basic documentation.

@NateBrady23
Copy link
Copy Markdown
Member

Great stuff @nmihajlovski! I will test this out locally by end of day tomorrow and get it in to round-14. This is definitely the gold standard of adding framework tests! Thanks again!

@nmihajlovski
Copy link
Copy Markdown
Contributor Author

Thanks, @nbrady-techempower, that sounds great! I am looking forward to seeing and comparing the high-level / low-level results.

@nmihajlovski
Copy link
Copy Markdown
Contributor Author

@nbrady-techempower Is there any reason why this PR hasn't been marked with any label (e.g. Round-14)?

@knewmanTE knewmanTE added this to the Round 14 milestone Sep 16, 2016
@NateBrady23
Copy link
Copy Markdown
Member

Looks like that's all set @nmihajlovski Lmk if you need anything else. I've had my head in some travis and toolset work.

@nmihajlovski
Copy link
Copy Markdown
Contributor Author

Thanks! :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants