LogDog: Update to archival V2.

common/api/logdog_coordinator/services/v1/service.proto

Index: common/api/logdog_coordinator/services/v1/service.proto
diff --git a/common/api/logdog_coordinator/services/v1/service.proto b/common/api/logdog_coordinator/services/v1/service.proto
index db1ef88b07954b804fcf935ac7874548aea11f2e..c1aa4fb11feb573eb5efd88563b920d4bbaf2054 100644
--- a/common/api/logdog_coordinator/services/v1/service.proto
+++ b/common/api/logdog_coordinator/services/v1/service.proto
@@ -78,7 +78,7 @@ message TerminateStreamRequest {
   // The log stream's secret.
   bytes secret = 2;
 
-  // The terminal index of the stream.
+  // The number of entries in the log stream.

[Vadim Sh., 2016/04/07 01:21:32]

in LogStreamState it is "index", here it is "number". What is it?

I mean in a list with 3 items, the terminal index is 2, but "number of entries"
is 3.

[dnj, 2016/04/11 17:20:04]

Sorry I was considering changing this, then reverted that decision, but I must
have forgotten to update the comment. Will fix.

   int64 terminal_index = 3;
 }
 
@@ -87,11 +87,13 @@ message TerminateStreamRequest {
 message ArchiveStreamRequest {
   // The path of the log stream that was archived.
   string path = 1;
-  // If true, the archive includes the full set of log stream entries between
-  // [0..terminal_index].
-  bool complete = 2;
+  // The number of log entries that were archived.
+  int64 log_entry_count = 2;
   // The highest log stream index that was archived.
   int64 terminal_index = 3;
+  // If true, there was an archival error, and the referenced streams should be
+  // considered invalid.
+  bool error = 4;

[Vadim Sh., 2016/04/07 01:21:32]

maybe "string error" // if not empty, there was an archival error...

to have some details about the error stored

[dnj, 2016/04/11 17:20:04]

I don't want to retain the error in datastore, so the only advantage this would
offer is to have the error be easily accessible in GAE logs. I suppose that's
worthwhile, though.

 
   // The archive URL of the log stream's stream data.
   string stream_url = 10;