LogDog/Archivist: Conditionally render data.

LogDog/Archivist: Conditionally render data.

When archiving log stream data, LogDog's Archivist supports emitting a
rendered data stream, which creates a flat file containing all of the
log stream's data. This is useful for direct consumption, but not
required. Because it doubles the amount of archival space required, it
is gated on two conditions:

- The project specifies that streams should always be rendered, or
- The stream requests to be rendered to binary by providing an
  extension.

Currently, these conditions are ignored, and the stream is always
rendered. This patch makes the Archivist actually pay attention to this
configuration and conditionally render the stream.

BUG=chromium:643243
TEST=unittest

Committed: https://github.com/luci/luci-go/commit/e1a482ae35e68cdbcb5385085e2d9a34219a50b8

[dnj, 2016-09-08 19:41:56]

dnj@chromium.org changed reviewers:
+ hinoka@chromium.org, iannucci@chromium.org

[dnj, 2016-09-08 19:41:56]

PTAL

[Ryan Tseng, 2016-09-08 19:54:22]

hinoka@google.com changed reviewers:
+ hinoka@google.com

[Ryan Tseng, 2016-09-08 19:54:22]

lgtm

[dnj, 2016-09-08 19:56:00]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2016-09-08 19:56:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-08 20:07:37]

Description was changed from

==========
LogDog/Archivist: Conditionally render data.

When archiving log stream data, LogDog's Archivist supports emitting a
rendered data stream, which creates a flat file containing all of the
log stream's data. This is useful for direct consumption, but not
required. Because it doubles the amount of archival space required, it
is gated on two conditions:

- The project specifies that streams should always be rendered, or
- The stream requests to be rendered to binary by providing an
  extension.

Currently, these conditions are ignored, and the stream is always
rendered. This patch makes the Archivist actually pay attention to this
configuration and conditionally render the stream.

BUG=chromium:643243
TEST=unittest
==========

to

==========
LogDog/Archivist: Conditionally render data.

When archiving log stream data, LogDog's Archivist supports emitting a
rendered data stream, which creates a flat file containing all of the
log stream's data. This is useful for direct consumption, but not
required. Because it doubles the amount of archival space required, it
is gated on two conditions:

- The project specifies that streams should always be rendered, or
- The stream requests to be rendered to binary by providing an
  extension.

Currently, these conditions are ignored, and the stream is always
rendered. This patch makes the Archivist actually pay attention to this
configuration and conditionally render the stream.

BUG=chromium:643243
TEST=unittest

Committed:
https://github.com/luci/luci-go/commit/e1a482ae35e68cdbcb5385085e2d9a34219a50b8
==========

[commit-bot: I haz the power, 2016-09-08 20:07:38]

Committed patchset #1 (id:1) as
https://github.com/luci/luci-go/commit/e1a482ae35e68cdbcb5385085e2d9a34219a50b8