Revise a comment

d5e073a4 · Shinya Maeda · c47a434c · d5e073a4
Commit d5e073a4 authored May 23, 2018 by Shinya Maeda
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

app/workers/rescue_stale_live_trace_worker.rb app/workers/rescue_stale_live_trace_worker.rb +3 -3

No files found.
--- a/app/workers/rescue_stale_live_trace_worker.rb
+++ b/app/workers/rescue_stale_live_trace_worker.rb
@@ -5,9 +5,9 @@ class RescueStaleLiveTraceWorker
  def perform
    # Reschedule to archive live traces
    #
-    # The target jobs are with the following conditions
+    # The targets are jobs with the following conditions
-    # - Finished 1 hour ago, but it has not had an acthived trace yet
+    # - It had been finished 1 hour ago, but it has not had an acthived trace yet
-    #   Jobs finished 1 hour ago should have an archived trace. Probably ArchiveTraceWorker failed by Sidekiq's inconsistancy
+    #   This case happens when sidekiq-jobs of archiving traces are lost in order to restart sidekiq instace which hit RSS limit
    Ci::BuildTraceChunk
      .include(EachBatch)
      .select(:build_id)