DataDog / agent-payload

Payload format description for communication between the agent and the Datadog backend
BSD 3-Clause "New" or "Revised" License
21 stars 19 forks source link

[protobuf] remove logs java protobuf and upgrade logs go compiler to non-legacy #346

Closed leeavital closed 1 month ago

leeavital commented 1 month ago

This generated java is no longer used, instead the logs java backend copies the protobuf file and regenerates using their own generator.

There are no differences between the legacy and modern proto compiler for Go, so we can upgrade to the latest protobuf compiler.

This is the final step before we can remove the legacy protobuf compiler.

What does this PR do?

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

Reviewers: please see the review guidelines.

leeavital commented 1 month ago

/merge

dd-devflow[bot] commented 1 month ago

:steam_locomotive: MergeQueue: pull request added to the queue

The median merge time in master is 2m.

Use /merge -c to cancel this operation!

leeavital commented 1 month ago

/merge -m squash

dd-devflow[bot] commented 1 month ago

:x: MergeQueue

PR already in the queue with status in_progress

If you need support, contact us on Slack #devflow with those details!