Initial distributor implementation

common/api/dm/service/v1/walk_graph.proto

 // Copyright 2016 The LUCI Authors. All rights reserved.
 // Use of this source code is governed under the Apache License, Version 2.0
 // that can be found in the LICENSE file.
 
 syntax = "proto3";
 
 import "google/protobuf/duration.proto";
 import "google/protobuf/timestamp.proto";
 
 import "graph_data.proto";
@@ skipping 92 matching lines @@
     bool attempt_result = 4;
 
     // ExpiredAttempts allows you to view attempts which have expired results,
     // which are normally excluded from the graph.
     bool expired_attempts = 5;
 
     // Executions is the number of Executions to include per Attempt. If this
     // is 0, then the execution data will be omitted completely.
     uint32 num_executions = 6;
 
+    // ExecutionInfoUrl fills in the Execution.DistributorInfo.Url value. This
+    // has some minor overhead, but is useful if you need to present clickable
+    // links for humans.
+    bool execution_info_url = 7;
+
     // FwdDeps instructs WalkGraph to include forward dependency information
     // from the result. This only changes the presence of information in the
     // result; if the query is walking forward attempt dependencies, that will
     // still occur even if this is false.
-    bool fwd_deps = 7;
+    bool fwd_deps = 8;
 
     // BackDeps instructs WalkGraph to include the backwards dependency
     // information. This only changes the presence of information in the result;
     // if the query is walking backward attempt dependencies, that will still
     // occur even if this is false.
-    bool back_deps = 8;
+    bool back_deps = 9;
+
+    // All is a shortcut that will cause all entries in this Include message to
+    // be set to true/max.
+    bool all = 128;

[dnj (Google), 2016/06/09 18:00:57]

(I don't like this. If you ever add new fields that you don't want to "all",
this could become weird. Plus, this runs the risk of lazy folk simply wasting
resources. I don't think it's unreasonable to ask people to just set the things
they want to "true".)

[iannucci, 2016/06/15 00:46:02]

Hm. Ok. I'll move it to a package level constant Message that can be copied into
place. This was mainly for dmtool's usage where the entire message was dumped
for logging/debugging reasons.

   }
   // Include allows you to add additional information to the returned
   // GraphData which is typically medium-to-large sized.
   Include include = 5;
 }