Re: Metrics updates in Release Notes

2018-10-31 Thread Roman Leventov
I think because Metrics / Event Emitting PRs usually make something that needs to be reflected in the release notes, while ordinary PRs - not so often. On Wed, 31 Oct 2018 at 18:47, Gian Merlino wrote: > Why is "metrics / event emitting" special, though? Why shouldn't we ask the > release

Re: Metrics updates in Release Notes

2018-10-31 Thread Gian Merlino
I don't think we have a doc about how to do a release, but yeah it would be great to have it. Dave, would you be able to put it together while you manage this release? I am sure it will differ substantially from what we've done in the past, because of the new Apache-ified stuff. On Wed, Oct 31,

Re: Metrics updates in Release Notes

2018-10-31 Thread Gian Merlino
Why not also tag those with "Release Notes"? It makes it a lot easier for release managers to do their jobs if they just have to look at one label. (Or two, I guess: "release notes" and "incompatible". But I would be down to merge them.) On Wed, Oct 31, 2018 at 9:26 AM David Lim wrote: > Thanks

Re: Metrics updates in Release Notes

2018-10-31 Thread David Lim
Thanks Roman. I'm helping with the release this time so I will check the PRs with that label and include them in the release notes as appropriate. As far as I know, there isn't any document like that, but I agree it would be quite useful. On Wed, Oct 31, 2018 at 9:04 AM Roman Leventov wrote: >

Metrics updates in Release Notes

2018-10-31 Thread Roman Leventov
It's suggested that the person that prepares Druid Release Notes (I think it's Jon usually) goes through all PRs labelled "Area - Metrics/Event Emitting" (