Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(647)

Unified Diff: common/api/dm/service/v1/walk_graph.proto

Issue 1537883002: Initial distributor implementation (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/luci-go@master
Patch Set: self review Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: common/api/dm/service/v1/walk_graph.proto
diff --git a/common/api/dm/service/v1/walk_graph.proto b/common/api/dm/service/v1/walk_graph.proto
index 4f9c0e34ae41c29ab106cda0d2e1751b2d6d32fb..48a8134776bf0203833cb1470b33ab2aa8dd500c 100644
--- a/common/api/dm/service/v1/walk_graph.proto
+++ b/common/api/dm/service/v1/walk_graph.proto
@@ -110,17 +110,26 @@ message WalkGraphReq {
// 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 tha
iannucci 2016/06/15 00:46:02 Hm. Ok. I'll move it to a package level constant M
}
// Include allows you to add additional information to the returned
// GraphData which is typically medium-to-large sized.

Powered by Google App Engine
This is Rietveld 408576698