[Findit] Datamodel change for triaging at CL level.

appengine/findit/model/wf_culprit.py

 # Copyright 2016 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 from google.appengine.ext import ndb
 
 from model import analysis_status as status
-from model.wf_suspected_cl import WfSuspectedCL
+from model.base_cl import BaseCL
 
 
-class WfCulprit(WfSuspectedCL):
+class WfCulprit(BaseCL):
   """Represents a culprit that causes a group of failures on Chromium waterfall.
 
   'Wf' is short for waterfall.
   """
+  # The list of builds in which the culprit caused some breakage.
+  builds = ndb.JsonProperty(indexed=False)

[lijeffrey, 2016/09/08 22:09:24]

so we indeed want this in both WfSuspectedCL and in WfCulprit too? Is there a
way to consolidate them?

[chanli, 2016/09/08 22:32:29]

For culprit a list of builds is enough, but we need a dict of information
related to each build in WfSuspectedCL...

 
   # When the code-review of this culprit was notified.
   cr_notification_time = ndb.DateTimeProperty(indexed=True)
 
   # The status of code-review notification: None, RUNNING, COMPLETED, ERROR.
   cr_notification_status = ndb.IntegerProperty(indexed=True)
 
   @property
   def cr_notification_processed(self):
     return self.cr_notification_status in (status.COMPLETED, status.RUNNING)
 
   @property
   def cr_notified(self):
     return self.cr_notification_status == status.COMPLETED
+
+  @classmethod
+  def Create(cls, repo_name, revision, commit_position):  # pragma: no cover
+    instance = cls(key=cls._CreateKey(repo_name, revision))
+    instance.repo_name = repo_name
+    instance.revision = revision
+    instance.commit_position = commit_position
+    instance.builds = []
+    return instance