https://bugs.ruby-lang.org/
https://bugs.ruby-lang.org/favicon.ico?1711330511
2011-02-21T22:17:12Z
Ruby Issue Tracking System
Ruby master - Feature #4415: Rubyのtest-allを並列化するパッチが完成しました
https://bugs.ruby-lang.org/issues/4415?journal_id=15902
2011-02-21T22:17:12Z
matz (Yukihiro Matsumoto)
matz@ruby.or.jp
<ul></ul><p>=begin<br>
まつもと ゆきひろです</p>
<p>In message "Re: <a href="/issues/4415">[ruby-dev:43226]</a> [Ruby 1.9-Feature#4415][Open] Rubyのtest-allを並列化するパッチが完成しました"<br>
on Mon, 21 Feb 2011 22:09:12 +0900, Shota Fukumori <a href="mailto:redmine@ruby-lang.org" class="email">redmine@ruby-lang.org</a> writes:</p>
<p>|Feature <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Rubyのtest-allを並列化するパッチが完成しました (Closed)" href="https://bugs.ruby-lang.org/issues/4415">#4415</a>: Rubyのtest-allを並列化するパッチが完成しました<br>
|<a href="http://redmine.ruby-lang.org/issues/show/4415" class="external">http://redmine.ruby-lang.org/issues/show/4415</a><br>
|<br>
|起票者: Shota Fukumori<br>
|ステータス: Open, 優先度: Normal<br>
|カテゴリ: lib<br>
|<br>
|# <a href="https://blade.ruby-lang.org/ruby-dev/43222">[ruby-dev:43222]</a> の続きです。経緯などはそちらを参照していただければと思います。<br>
|<br>
|パッチが完成したのでチケットを作成しました。<br>
|<br>
|まつもとさんは<a href="https://blade.ruby-lang.org/ruby-dev/43224">[ruby-dev:43224]</a>で「バグがとれたら入れちゃったら?」と言っていました。<br>
|<br>
|一部テストについては改変していますが、それについては取りこまなくてもFailになることは無いと思います。<br>
|(workerで失敗したテストは並列では無く実行をしなおす為。ただし若干速度に影響がでるかと)<br>
|<br>
|では、コミットをよろしくお願いします。</p>
<p>コミット権も差し上げましょう。希望するアカウント名、PGP署名<br>
したssh2公開鍵を <a href="mailto:cvs-admin@ruby-lang.org" class="email">cvs-admin@ruby-lang.org</a> に送ってください。</p>
<p>久々の最年少更新か。</p>
<p>=end</p>
Ruby master - Feature #4415: Rubyのtest-allを並列化するパッチが完成しました
https://bugs.ruby-lang.org/issues/4415?journal_id=15903
2011-02-21T22:23:12Z
kou (Kouhei Sutou)
kou@cozmixng.org
<ul></ul><p>=begin<br>
須藤です。</p>
<p>In <a href="mailto:4d6263f6f161d_1dc2aafe934324d4@redmine.ruby-lang.org" class="email">4d6263f6f161d_1dc2aafe934324d4@redmine.ruby-lang.org</a><br>
"<a href="/issues/4415">[ruby-dev:43226]</a> [Ruby 1.9-Feature#4415][Open] Rubyのtest-allを並列化するパッチが完成しました" on Mon, 21 Feb 2011 22:09:12 +0900,<br>
Shota Fukumori <a href="mailto:redmine@ruby-lang.org" class="email">redmine@ruby-lang.org</a> wrote:</p>
<blockquote>
<p>パッチが完成したのでチケットを作成しました。</p>
<p>まつもとさんは<a href="https://blade.ruby-lang.org/ruby-dev/43224">[ruby-dev:43224]</a>で「バグがとれたら入れちゃったら?」と言っていました。</p>
<p>一部テストについては改変していますが、それについては取りこまなくてもFailになることは無いと思います。<br>
(workerで失敗したテストは並列では無く実行をしなおす為。ただし若干速度に影響がでるかと)</p>
<p>では、コミットをよろしくお願いします。</p>
</blockquote>
<p>水を差すようで申し訳ないのですが、a, b, xなどの名前に意味の<br>
ない変数名を使わないようにするなど、もう少し整理した方がよい<br>
気がします。</p>
<blockquote>
<ul>
<li>
<pre><code> def jobs_status
</code></pre>
</li>
<li>
<pre><code> return unless @opts[:job_status]
</code></pre>
</li>
<li>
<pre><code> puts "" unless @opts[:verbose]
</code></pre>
</li>
<li>
<pre><code> if @opts[:job_status]
</code></pre>
</li>
<li>
<pre><code> b = []
</code></pre>
</li>
<li>
<pre><code> str = @workers.map { |x|
</code></pre>
</li>
<li>
<pre><code> a = "#{x[:pid]}:#{x[:status].to_s.ljust(7)}"
</code></pre>
</li>
<li>
<pre><code> if x[:file]
</code></pre>
</li>
<li>
<pre><code> if @opts[:job_status_type] == :replace
</code></pre>
</li>
<li>
<pre><code> a = "#{x[:pid]}=#{x[:file]}"
</code></pre>
</li>
<li>
<pre><code> else
</code></pre>
</li>
<li>
<pre><code> if a.size > x[:file].size
</code></pre>
</li>
<li>
<pre><code> b << x[:file].ljust(a.size)
</code></pre>
</li>
<li>
<pre><code> else
</code></pre>
</li>
<li>
<pre><code> a << " "*(x[:file].size-a.size)
</code></pre>
</li>
<li>
<pre><code> b << x[:file]
</code></pre>
</li>
<li>
<pre><code> end
</code></pre>
</li>
<li>
<pre><code> end
</code></pre>
</li>
<li>
<pre><code> else
</code></pre>
</li>
<li>
<pre><code> b << " "*a.size
</code></pre>
</li>
<li>
<pre><code> end
</code></pre>
</li>
<li>
<pre><code> a
</code></pre>
</li>
<li>
<pre><code> }.join(" ")
</code></pre>
</li>
</ul>
</blockquote>
<p>=end</p>
Ruby master - Feature #4415: Rubyのtest-allを並列化するパッチが完成しました
https://bugs.ruby-lang.org/issues/4415?journal_id=15904
2011-02-21T22:27:29Z
sorah (Sorah Fukumori)
her@sorah.jp
<ul></ul><p>=begin<br>
あ、ありがとうございます!</p>
<p>これから出てくるバグに備えて、lib/test/*のメンテナにも立候補したいと思います。</p>
<p>2011/2/21 Yukihiro Matsumoto <a href="mailto:matz@ruby-lang.org" class="email">matz@ruby-lang.org</a>:</p>
<blockquote>
<p>コミット権も差し上げましょう。希望するアカウント名、PGP署名<br>
したssh2公開鍵を <a href="mailto:cvs-admin@ruby-lang.org" class="email">cvs-admin@ruby-lang.org</a> に送ってください。</p>
<p>久々の最年少更新か。</p>
</blockquote>
<p>--<br>
Shota Fukumori a.k.a. @sora_h - <a href="http://codnote.net/" class="external">http://codnote.net/</a></p>
<p>=end</p>
Ruby master - Feature #4415: Rubyのtest-allを並列化するパッチが完成しました
https://bugs.ruby-lang.org/issues/4415?journal_id=15905
2011-02-21T22:33:11Z
sorah (Sorah Fukumori)
her@sorah.jp
<ul></ul><p>=begin<br>
ふむ。では修正してから取りこみます。</p>
<p>2011/2/21 Kouhei Sutou <a href="mailto:kou@cozmixng.org" class="email">kou@cozmixng.org</a>:</p>
<blockquote>
<p>水を差すようで申し訳ないのですが、a, b, xなどの名前に意味の<br>
ない変数名を使わないようにするなど、もう少し整理した方がよい<br>
気がします。</p>
</blockquote>
<p>--<br>
Shota Fukumori a.k.a. @sora_h - <a href="http://codnote.net/" class="external">http://codnote.net/</a></p>
<p>=end</p>
Ruby master - Feature #4415: Rubyのtest-allを並列化するパッチが完成しました
https://bugs.ruby-lang.org/issues/4415?journal_id=15919
2011-02-22T12:37:07Z
sorah (Sorah Fukumori)
her@sorah.jp
<ul><li><strong>Status</strong> changed from <i>Open</i> to <i>Closed</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>=begin<br>
This issue was solved with changeset r30939.<br>
Shota, thank you for reporting this issue.<br>
Your contribution to Ruby is greatly appreciated.<br>
May Ruby be with you.</p>
<hr>
<ul>
<li>lib/test/unit.rb: Add new options; --jobs,-j,--ruby,--jobs-status,<br>
--no-retry.<br>
[Feature <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Rubyのtest-allを並列化するパッチが完成しました (Closed)" href="https://bugs.ruby-lang.org/issues/4415">#4415</a>] <a href="/issues/4415">[ruby-dev:43226]</a>,<a href="https://blade.ruby-lang.org/ruby-dev/43222">[ruby-dev:43222]</a>,<a href="https://blade.ruby-lang.org/ruby-core/35294">[ruby-core:35294]</a></li>
<li>lib/test/unit/parallel.rb: Used at test/unit --jobs(-j) option.</li>
<li>test/csv/test_serialization.rb: test/unit parallel running ready.</li>
<li>test/rake/test_file_task.rb: test/unit parallel running ready.<br>
=end</li>
</ul>