ruby-list

こんにちは、金井です。

MoonWolf <moonwolf@moonwolf.com> wrote:
> C言語版にこだわらなければ、PureRuby版のStringIOでもいいと思います。

ありがとうございます。さっそく試してみました。

FORM中には次の<INPUT>があります。合計 210個です。
　HIDDEN:2 SUBMIT:2 TEXT:41 CHECKBOX:134 SELECT:29 FILE:2
２つのFILEは 6番目,210番目(すなわち最後)にあります。
FILE以外には若干の入力があり、CONTENT_LENGTH は 20942 です。
FILEとしては約54.5KBのJPGを使いました。
表の中の時間の単位は ms です。
計測のたびにread_multipart時間にして200ms程度のばらつきがありました。

                         read_multipart  Tempfile.new  StrionIO.new
   StringIO  FILE1 FILE2           時間  時間/回数     時間/回数
-------------------------------------------------------------------
1-1   ×      ×   ×              7531  7046/210         0/  0
1-2   ×      ○   ×              7515  7062/210         0/  0
1-3   ×      ×   ○              7547  6938/210         0/  0
-------------------------------------------------------------------
2-1   ○      ×   ×              2860  2438/103       141/107
2-2   ○      ○   ×              1703  1249/ 56        78/154
2-3   ○      ×   ○              7391  6964/210         0/  0
-------------------------------------------------------------------
3-1   ○      ×   ×               360     0/  0        32/210
3-2   ○      ○   ×               610   141/  6        62/204
3-3   ○      ×   ○              7469  7001/210         0/  0
-------------------------------------------------------------------

2-1を例に読み方を説明します。
　stringio.rbが存在する。
　２つの<INPUT type="file">のうちの最初のものにだけファイルを指定。
　read_multipart関数の所要時間が2860ms。
　その中でTempfile.newが103回あり、その合計所要時間が2438ms。
　StringIO.newが107回あり、その合計所用時間が141ms。

さて、
表の最初の３行は StringIO 無しの場合です。
FILEが 両方空白/最初のだけ指定/最後のだけ指定 した場合の３通りです。
read_multipartの所要時間のほとんどが 210回の Tempfile.new に費や　
されているのが分かります。平均するとだいたい 33ms くらいでしょうか。

次の３行は StringIO 有りの場合です。劇的に速くなっています。
ただ2-3から分かるように、最後の最後にFILEがある場合はStringIOの
御利益が無くなってしまいます。

最後の３行は、Tempfileを使うかStringIOを使うかの判定をしている
　if 10240 < content_length
という行を、試しに
　if 10240*3 < content_length
に変えてみた場合です。
3-1を見れば分かりますが、CONTENT_LENGTHは20942ですので、Tempfile.new
は結局１回も実行されません。

以上のように、FILEをFORMの後ろの方に置きさえしなければStringIOの効果は
絶大です。この制約もread_multipart関数に手を入れれば解消できるような
気がしますが、今の私の手には余りますので cgi.rb はそのままで stringio
を入れるだけで運用しようと思います。

どうもありがとうございました。


MoonWolf <moonwolf@moonwolf.com> wrote:
> この程度のFORMで遅くなるのはcgi.rbにも原因があります。
> read_multipart中の以下の部分は無駄が多いです。
> 
> >           begin
> >             require "stringio" if not defined? StringIO
> >             body = StringIO.new
> >           rescue LoadError
> >             require "tempfile"
> >             body = Tempfile.new("CGI")
> >           end
> メソッド実行時ではなく、ライブラリロード時に解決すべき問題です。

　律速段階は Tempfile.new であることが分かりましたので、処理時間に関し
ては、あまり効果がないのではないかと思います。

--
金井 隆 (KANAI Takashi)
kanai@nn.iij4u.or.jp

Thread

Prev Next

In This Thread

Prev Next

[#35685] Exerb 2.0.0 動きません — AOKI Yoshihiro <aoki@...>

[#35688] サブクラスのオブジェクト生成時に、スーパークラスの初期化を行うには ? — Onodera Takafumi <onodera-tak@...>

[#35690] Ruby/SDL がうまく動きません — Mitsuhiko_Tomomi <mikkun_ota@...>

[#35713] Selection IPv4/IPv6 at TCPSocket — NISHI Takao <zophos@...9.com>

[#35715] すべてのメソッドを（最小コストで）フックしたい — "井上 浩一" <kyoui32@...>

[#35716] すべてのメソッドを最小コストでフックしたい。 — "井上 浩一" <kyoui32@...>

[#35718] Re: すべてのメソッドを最小コストでフックしたい。 — "井上 浩一" <kyoui32@...>

[#35722] パターンマッチした値を保持する変数は — goto@...

[#35729] SMTP サーバクラス — とみたまさひろ <tommy@...>

[#35730] how to share some value through parent class's method — YANAGAWA Kazuhisa <kjana@...4lab.to>

[#35733] tail recursion elimination — Takaaki Tateishi <ttate@...>

[#35737] Win32 環境での PStore の動かし方 — fukumit <fukumit@...>

[#35743] FortranArray, yet another handler of array — NISHIMATSU Takeshi <t-nissie@...>

[#35745] Ruby vs. Expect — T_A_K_E <koichi.takehara@...>

[#35748] DBD::OCI8 based on Ruby/OCI8 — KUBO Takehiro <kubo@...>

[#35754] [ANN] SWIG 1.3.14 リリース — Shibukawa Yoshiki <yoshiki@...>

[#35759] Bignum fast multiplication — IKEGAMI Daisuke <daisu-ik@...>

[#35769] [ANN] SWIG-Ruby ドキュメント翻訳 — Shibukawa Yoshiki <yoshiki@...>

[#35770] cgi.rb の form — rio-t@...

[#35772] Unsecure world writeable dirの警告 — "井上 浩一" <kyoui32@...>

[#35789] multipart な CGI を速くしたい — Takashi Kanai <kanai@...4u.or.jp>

[#35792] "self = / ー /" でエラー — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#35795] Re: "self = / ー /" でエラー — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#35799] Re: "self = / ー /" でエラー — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#35810] [ANN] RubyCocoa 0.2.7 — FUJIMOTO Hisakuni <hisa@...>

[#35811] 祝 1.7.3 But cast_256.so Segmentaation fault — "Inoue" <rubyist@...1.117.ne.jp>

[#35813] — NAWATE Masahiko <agul@...>

[#35816] [ANN] irbsh-0.8.0 — rubikitch <rubikitch@...>

[#35817] CGI::pretty — rio-t@...

[#35821] [ANN] RDE0.9.7.0 — sakazuki <QZS01353@...>

[#35822] profile.rb の使い方 — rio-t@...

[#35823] [Q]UNIX command in Ruby — Masaaki Sakano <mas@...>

[#35829] cygwin 版 ruby で ARGV に日本語 — rio-t@...

[ruby-list:35812] Re: multipart な CGI を速くしたい

Thread

In This Thread

[#35715] すべてのメソッドを（最小コストで）フックしたい — "井上浩一" <kyoui32@...>

[#35716] すべてのメソッドを最小コストでフックしたい。 — "井上浩一" <kyoui32@...>

[#35718] Re: すべてのメソッドを最小コストでフックしたい。 — "井上浩一" <kyoui32@...>

[#35772] Unsecure world writeable dirの警告 — "井上浩一" <kyoui32@...>