Skip to content

Commit

Permalink
refactor: readme.md
Browse files Browse the repository at this point in the history
* add graphs for different setups
* fix results file
  • Loading branch information
karatakis committed Nov 6, 2024
1 parent 8276dba commit 0ebf750
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 7 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/bench.yml
Original file line number Diff line number Diff line change
Expand Up @@ -142,8 +142,8 @@ jobs:
ref: ${{ github.event_name == 'pull_request_target' && github.event.pull_request.head.sha || github.sha }}
- name: Collect results table
run: |
echo "### [Small Payload - 362 bytes](./source/small.json)"
echo "| Server | Status | RPS | Latency |" > "results.md"
echo "### [Small Payload - 362 bytes](./source/small.json)" >> "results.md"
echo "| Server | Status | RPS | Latency |" >> "results.md"
echo "| ---: | ---: | ---: | ---: |" >> "results.md"
echo "| [Nginx](https://nginx.org/en/) | \
${{ fromJson(needs.read.outputs.result).has_error.small_nginx_graphql }} | \
Expand Down Expand Up @@ -181,8 +181,8 @@ jobs:
\`${{ fromJson(needs.read.outputs.result).p95.small_wundergraph_default }} sec\` |" >> "results.md"
echo "### [Medium Payload - 12,598 bytes](./source/medium.json)"
echo "| Server | Status | RPS | Latency |" > "results.md"
echo "### [Medium Payload - 12,598 bytes](./source/medium.json)" >> "results.md"
echo "| Server | Status | RPS | Latency |" >> "results.md"
echo "| ---: | ---: | ---: | ---: |" >> "results.md"
echo "| [Nginx](https://nginx.org/en/) | \
${{ fromJson(needs.read.outputs.result).has_error.medium_nginx_graphql }} | \
Expand Down Expand Up @@ -220,8 +220,8 @@ jobs:
\`${{ fromJson(needs.read.outputs.result).p95.medium_wundergraph_default }} sec\` |" >> "results.md"
echo "### [Big Payload - 112,838 bytes](./source/big.json)"
echo "| Server | Status | RPS | Latency |" > "results.md"
echo "### [Big Payload - 112,838 bytes](./source/big.json)" >> "results.md"
echo "| Server | Status | RPS | Latency |" >> "results.md"
echo "| ---: | ---: | ---: | ---: |" >> "results.md"
echo "| [Nginx](https://nginx.org/en/) | \
${{ fromJson(needs.read.outputs.result).has_error.big_nginx_graphql }} | \
Expand Down
4 changes: 3 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,9 @@ This document presents a comparative analysis of several renowned GraphQL Federa
## Architecture

![image info](./files/diagram.png)
| Big Payload | Medium Payload | Small Payload |
| --- | --- | --- |
| ![image info](./files/diagram_big.png) | ![image info](./files/diagram_medium.png) | ![image info](./files/diagram_small.png) |

### Components

Expand Down
Binary file removed files/diagram.png
Binary file not shown.
Binary file added files/diagram_big.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added files/diagram_medium.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added files/diagram_small.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit 0ebf750

Please sign in to comment.