Update documentation for TimeInterval

[Marco-Premier]

• Update documentation for TimeInterval
• Add oneof type to ModelRollout

[wfa-reviewable]

This change is 

[SanjayVas]

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @jojijac0b and @Marco-Premier)

---

src/main/proto/wfa/measurement/api/v2alpha/model_rollout.proto line 47 at r1 (raw file):

    // 'end_time' cannot precede 'start_time'.
    TimeInterval gradual_rollout_period = 8 [
      (google.api.field_behavior) = REQUIRED,

This field itself is not required. Rather, the oneof itself is required. (There is no way to document this with an annotation. Just use a regular comment on the oneof field.

Code quote:

(google.api.field_behavior) = REQUIRED,

[Marco-Premier]

Reviewable status: 1 of 2 files reviewed, 1 unresolved discussion (waiting on @jojijac0b and @SanjayVas)

---

src/main/proto/wfa/measurement/api/v2alpha/model_rollout.proto line 47 at r1 (raw file):

Previously, SanjayVas (Sanjay Vasandani) wrote…

This field itself is not required. Rather, the oneof itself is required. (There is no way to document this with an annotation. Just use a regular comment on the oneof field.

Correct, thanks. I marked it as IMMUTABLE as well as instant_rollout_time

[SanjayVas]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @jojijac0b)