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

Useable http2 clients #11

Merged
merged 2 commits into from Jun 24, 2018

Conversation

Projects
None yet
2 participants
@Goryudyuma
Copy link
Collaborator

Goryudyuma commented Jun 21, 2018

#10
の4を実装してみた。
clientを複数個用意して、requestの時にランダムでどれかから送るようにしている。
admin/benchmarker/benchmark.go にworkload*5としていて、手元の環境ではhttp/2にする事で得点が約1.25倍になる。
ちなみにworkload*1だと得点が1.5倍くらいに伸びてしまい、幾ら何でもやりすぎだと思った。

Goryudyuma added some commits Jun 21, 2018

ベンチマークの最初にclientを生成する
現在、workloadの数*5のclientを生み出している。http2に変える事により、得点は1.25倍くらいになる。
@showwin

This comment has been minimized.

Copy link
Owner

showwin commented Jun 23, 2018

4番はclient pool に [http1.1, http2(毎回新規作成), http2(継続)] なやつたちが入っていて、そこからランダムに取られる感じかと思っていたけど、
最初にたくさんclientを作っておいて、全部それを使い回す感じか!

手元の環境ではhttp/2にする事で得点が約1.25倍になる。

は workload いくつで計測しました?
workload 数が少ない方が新規でセッション張る回数が少なくなるからスコアが workload 5 > workload 30 とかになる可能性があると思うのだけど、ぼくが実装勘違いしているかな… 💭

@Goryudyuma

This comment has been minimized.

Copy link
Collaborator Author

Goryudyuma commented Jun 23, 2018

もう一度確認してみます。

workloadは単に一定数の投票者の集まりで、例えばworkload*3とworkload*30は投票者の人数が10倍多いという仮定でした(①)。
そこでclientをworkloadに比例して増やすこの実装は、clientは投票所にある端末と仮定して、人数あたりの投票所の数は一定かなと思って作りました。
それとも、投票者一人あたりに同じセッションが使われる可能性があるのは、3と30では1/10になるという視点からでしょうか?(②)

workload 5 > workload 30となるのは、①か②のどちらかかな〜と思ったのですが、詳しくそう思った理由を教えてください〜

@showwin

This comment has been minimized.

Copy link
Owner

showwin commented Jun 23, 2018

あー理解した!

workload3とworkload30は投票者の人数が10倍多いという仮定でした(①)。

を忘れていました。

であれば、workload 5 > workload 30 になる可能性はないと思うので、大丈夫です!

@showwin

This comment has been minimized.

Copy link
Owner

showwin commented Jun 23, 2018

手元の環境ではhttp/2にする事で得点が約1.25倍になる。

ぐらいが適切だと思います!
LGTM 👍

@showwin showwin self-requested a review Jun 23, 2018

@Goryudyuma

This comment has been minimized.

Copy link
Collaborator Author

Goryudyuma commented Jun 23, 2018

一応確認のため、手元の環境で、

  1. workloadが3/30
  2. http2あり、無し
  3. Python初期実装、Goのそこそこスコアが出るやつ

の、2*2*2の8回のスコアを取ってみましたが、workload 5 > workload 30 とはならなさそうでした!

@showwin showwin merged commit cacb100 into showwin:master Jun 24, 2018

@Goryudyuma Goryudyuma deleted the Goryudyuma:useable-http2-clients branch Jun 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.