[GitHub] [skywalking-data-collect-protocol] fgksgf commented on a diff in pull request #70: Add layer to event

[GitBox <gi...@apache.org>]

fgksgf commented on code in PR #70:
URL: https://github.com/apache/skywalking-data-collect-protocol/pull/70#discussion_r845869676


##########
event/Event.proto:
##########
@@ -65,6 +65,9 @@ message Event {
   // The end time (in milliseconds) of the event. , measured between the current time and midnight, January 1, 1970 UTC.
   // This field may be empty if the event has not stopped yet, otherwise it should be a valid timestamp after `startTime`.
   int64 endTime = 8;
+  
+  // Name of the layer to which the event belongs. If absent, the OAP would set `layer`=`ID: 0, NAME: undefined`.

Review Comment:
   > I think undefined is meaningless. Do you expect to ignore no-layer event?
   
   Nope, how about `general`?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org