ec-cube icon indicating copy to clipboard operation
ec-cube copied to clipboard

Published 20 hours ago • verified publisher icon EC-CUBE

アパッチベンチで接続速度計算について

Open sai-gillingham opened this issue 2 years ago • 6 comments

概要(Overview・Refs Issue)

Pattern DB Version
Base Install Result PGSQL 4.1
Base Install Result PGSQL 4.2
Base Install Result MYSQL 4.1
Base Install Result MYSQL 4.2
10000 Product Result (30 Category + 3x3 product class (チョコ, バニラ、抹茶 x 16x16, 32x32, 64x64) PGSQL 4.1
10000 Product Result (30 Category + 3x3 product class (チョコ, バニラ、抹茶 x 16x16, 32x32, 64x64) PGSQL 4.2
10000 Product Result (30 Category + 3x3 product class (チョコ, バニラ、抹茶 x 16x16, 32x32, 64x64) MYSQL 4.1
10000 Product Result (30 Category + 3x3 product class (チョコ, バニラ、抹茶 x 16x16, 32x32, 64x64) MYSQL 4.2

上記のそれぞれの環境で ホームページと 商品一覧の接続速度を計算する。

※　https://github.com/EC-CUBE/ec-cube/pull/5434　 のブランチにあるコミットが多いため、別のブランチを作って、全てのコミットをまとめました。

方針(Policy)

.github アクションで それぞれの環境を作って、 5回、 ab -n 100 -c 10 -l http://localhost:8080/products/list ab -n 100 -c 10 -l http://localhost:8080/ 上記のコマンドを実行して、 abの接続速度の平均値を取得して csvに保存する処理です。

実装に関する補足(Appendix)

参考のため、私の.githubアクションのフローとgithub アクションが作ったartifactを送ります。 https://github.com/sai-gillingham/ec-cube/actions/runs/2646939990

テスト（Test)

相談（Discussion）

マイナーバージョン互換性保持のための制限事項チェックリスト

  • [ ] 既存機能の仕様変更はありません
  • [ ] フックポイントの呼び出しタイミングの変更はありません
  • [ ] フックポイントのパラメータの削除・データ型の変更はありません
  • [ ] twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • [ ] 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

  • [ ] 動作確認
  • [ ] コードレビュー
  • [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • [ ] 互換性が保持されているか
  • [ ] セキュリティ上の問題がないか
    • [ ] 権限を超えた操作が可能にならないか
    • [ ] 不要なファイルアップロードがないか
    • [ ] 外部へ公開されるファイルや機能の追加ではないか
    • [ ] テンプレートでのエスケープ漏れがないか

sai-gillingham avatar Sep 05 '22 10:09 sai-gillingham

Codecov Report

Merging #5768 (9862355) into 4.2 (be93b95) will increase coverage by 0.71%. The diff coverage is n/a.

:exclamation: Current head 9862355 differs from pull request most recent head 3b76f80. Consider uploading reports for the commit 3b76f80 to get more accurate results

@@             Coverage Diff              @@
##                4.2    #5768      +/-   ##
============================================
+ Coverage     78.16%   78.87%   +0.71%     
+ Complexity     6347     6346       -1     
============================================
  Files           473      473              
  Lines         21425    21327      -98     
============================================
+ Hits          16747    16822      +75     
+ Misses         4678     4505     -173
Flag Coverage Δ
E2E 64.84% <ø> (+0.79%) :arrow_up:
Unit 77.82% <ø> (-0.01%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/Eccube/Entity/Customer.php 91.95% <0.00%> (+0.45%) :arrow_up:
src/Eccube/Kernel.php 89.74% <0.00%> (+0.57%) :arrow_up:
src/Eccube/Entity/ProductClass.php 92.70% <0.00%> (+0.67%) :arrow_up:
src/Eccube/Entity/Category.php 76.00% <0.00%> (+0.75%) :arrow_up:
src/Eccube/Entity/Order.php 92.20% <0.00%> (+0.76%) :arrow_up:
src/Eccube/Entity/Member.php 85.56% <0.00%> (+0.87%) :arrow_up:
src/Eccube/Repository/OrderRepository.php 95.26% <0.00%> (+1.33%) :arrow_up:
...e/Doctrine/ORM/Mapping/Driver/AnnotationDriver.php 73.33% <0.00%> (+1.59%) :arrow_up:
src/Eccube/Repository/CustomerRepository.php 98.72% <0.00%> (+1.85%) :arrow_up:
src/Eccube/Form/Extension/DoctrineOrmExtension.php 74.35% <0.00%> (+1.85%) :arrow_up:
... and 26 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov-commenter avatar Sep 05 '22 11:09 codecov-commenter

@sai-gillingham Requests per secondの数値がほしいのですが修正可能でしょうか

chihiro-adachi avatar Sep 05 '22 22:09 chihiro-adachi

@sai-gillingham Requests per secondの数値がほしいのですが修正可能でしょうか

了解いたしました。今からRequests per secondの数値をgithub actionsのArtifactsに入れます。

sai-gillingham avatar Sep 06 '22 01:09 sai-gillingham

@sai-gillingham ありがとうございます！

chihiro-adachi avatar Sep 06 '22 01:09 chihiro-adachi

@chihiro-adachi Requests per second数値をArtifactsに入れました。 https://github.com/sai-gillingham/ec-cube/actions/runs/2997488309

Artifactsのcsvファイルの構成は下記になりますがよろしいでしょうか？

Pattern DB Page 4.1.2 - TIME PER REQUEST 4.2 - TIME PER REQUEST 4.1.2 - REQUESTS PER SECOND 4.2 - REQUESTS PER SECOND
Base Install Result PGSQL TOP 33.1154 35.082 30.28 28.584
Base Install Result PGSQL LIST 45.885 48.4906 21.81 20.628
Base Install Result MYSQL TOP 40.6144 26.1696 25.656 40.768
Base Install Result MYSQL LIST 51.8228 32.8474 19.332 30.504
10000 Product Result PGSQL TOP 36.2762 44.1284 28.472 23.572
10000 Product Result PGSQL LIST 148.2864 178.883 6.748 5.592
10000 Product Result MYSQL TOP 37.6494 29.3138 27.704 36.12
10000 Product Result MYSQL LIST 186.0892 146.7224 5.394 6.826

sai-gillingham avatar Sep 06 '22 04:09 sai-gillingham

@sai-gillingham はい、大丈夫です

chihiro-adachi avatar Sep 06 '22 05:09 chihiro-adachi