-
Notifications
You must be signed in to change notification settings - Fork 670
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
Create new types of tests #7732
Comments
Hey, Thanks for this proposal. Seems to go in a direction that I'm confortable with I suggest slight enhancement.
|
Can you make please a message with description of new endpoints and the plan when they will be enabled in testing? |
Not sure to get you @cyrusmsk |
As I understood you want to introduce new endpoints for testing? With sleep with json maybe with SQLite. |
The idea is to make new tests, so as this project could be meaningul. But we plan to do this incrementally, no timeline defined at this time |
In that case maybe new filter field will be helpful. Something like "Old tests" - "New tests". So in the result will be possible to see different implementations without any breaking changes. |
You mean add this on UI If you make somes PRs why not ... Just a joke I'm more on the line of having a new UI when all tests are ready, and in the meantime we can just make the backend code step by step |
We are going to write new test (add some endpoints)
I plan to use a real sqlite database
any thought @whiplash @cyrusmsk @ahopkins @appleboy @dalisoft @sy-records @System-Glitch @pimbrouwers @panesofglass @0xTim @leocavalcante @lukaszlenart ? |
cc @mario-huang |
Many frameworks don't have anything related to the features you want to test. It would essentially be testing the standard library. I'm not sure those would be very valuable. I thinking mainly of:
|
I think
At best, if those tests could be run at same LAN/network but over 10GbE network not at same machine as running server Here another benchmark alternative to TechEmpower and this repo - https://github.com/SaltyAom/bun-http-framework-benchmark |
so for you @System-Glitch
I'll argue that this is what we have mainly in server-side API, thus necessary to implement. for you @dalisoft /external-service is not necessary, but I can argue that some API could use any external service (the first one is a distant db) |
Do all frameworks will require to implement all tests? or they can skip some of them to be presented in the table? |
Yes, the new ones
Not thought about it. But could be intersting to have an customer choice to pick 1 endpoint or all |
@dalisoft @System-Glitch I think the project should decide on the scope.. The name of the repo is "web-frameworks" so probably it should be more than web-server functionality, but maybe not necessary "full stack" frameworks. Especially most of the presented solutions in the repo are web-servers/micro frameworks. So there could be 2 types of tests for micro and full stack frameworks and 2 tables. @waghanza also recently I watched the video (https://www.youtube.com/watch?v=qB422eeOFr8) and we can leverage on the ideas from this test. One of the nice idea is the size of the headers in request. Sending small request for Hello world - not really relevant to real use cases. Maybe only in some micro-service architecture (but probably they will use something like gPRC in that case). |
Hi @cyrusmsk , Thanks for your ideas. If I summarize their is web server things and web framework things, and you suggest to split in two tables
Finally, I think that in order not to be only for tech plumers, we need to use the generic term of web framework. I agree that their is micro/full (and even nano, for some kind), but this is more for experts. The ultimate goal of this projects is to help choosing your stack (here server-side) when migrating an existing tool / crafting a new product. Having this distingtion, even if it exists, will lead to present complex results to analyze by a non tech-expert. The idea is to show the average of all endpoints by default, and allow fine customization (bu this is for the UI part, in step2). The question is : in a context of a server-side api, will it be relevant to use a real local sqlite database in here ? PS : I'll start to implement this in a few weeks if relevant |
Also, with the simplest scenario we have, we can see that go is not automatically better (in terms of performance) than bun but the idea to have those endpoints is to have more reallistic workload, and I think this order will change 😛 |
I think local sqlite could be fine. Or maybe even NoSQL solution that could be used to store some cache data. I think another questions is to send not only 1 request many times, but also have an ability to send requests from the list of available one. Something line '/users/user_data={1,2,3,4}' or even send some data in JSON format as a payload. Which is quite common scenario for APIs. |
Good idea for multiple, but maybe in a second time (to huge to change so many things right now) Go local sqlite then (or any other tool) |
Hello,
Currently, the web-framework benchmark tests an empty endpoint.
However, web frameworks are about more than just request delay; they are mainly about processing complex tasks like JSON serialization, object composition, DB requests, etc.
Have you ever considered enriching the benchmark with new types of tests? It could provide a more comprehensive evaluation of web frameworks.
Imagine the following endpoints:
/ - the current behavior, we check the response time
/external-service - we call external services (a function that does a sleep)
/serialization - we send and receive a large JSON body in POST request with few query parameters
/http-client - we test the standard HTTP client and load a large file hosted in the server itself
/web - we do complete web scenarios with JSON in requests, DB requests, data transformations, and large JSON in response
We also deprecated the endpoints :
/user/{id}
/user
Because they do not bring value compared to the / endpoint
The current benchmark does not need to be updated. But each new piece of code that is not a framework update must implement that logic.
The text was updated successfully, but these errors were encountered: