title: protobuf embedded message best practice
body: <p>I have a protobuf message that contains another repeating message. It looks like this:</p>
  
  <pre><code>message AddGroupsRequest
  {
      message GroupProperties
      {
          string name = 1;
          int32 some_property_1 = 2;
          int32 some_property_2 = 3;
      }
  
      repeated GroupProperties group_properties = 1;
  }
  </code></pre>
  
  <p>My question is whether that's the best way to encode the messages, or should they be separated like this:</p>
  
  <pre><code>message GroupProperties
  {
      string name = 1;
      int32 some_property_1 = 2;
      int32 some_property_2 = 3;
  }
  
  message AddGroupsRequest
  {
      repeated GroupProperties group_properties = 1;
  }
  </code></pre>
  
  <p>Are there implications for defining the message one way or another?</p>
  
  <p>Thanks</p>