Hi all,

It seems our PR description template is a bit outdated, and I would like to
propose updating it.

I was working on a Kubernetes related PR, and realized that our PR
description does not mention the new Kubernetes integration questioning
about deployment related changes. Currently is is as follows:

> Anything that affects deployment or recovery: JobManager (and its
> components), Checkpointing, Yarn/Mesos, ZooKeeper:
>

In addition to outdated contents, there might be other stuff that we want
to add to the template. For example, I would suggest add a question about
whether there are any memory allocation introduced by the PR, so we review
them carefully and avoid problems due to un-accounted memory allocations
like FLINK-15981. (To be fair, for FLINK-15981 the memory allocation was
introduced before we start to account for everything memory usage, but
noticing such memory allocations early should help us prevent similar
problems in the future).

Therefore, I'd also like to collect ideas on how do you think the template
should be updated in this discussion thread.

Looking forward to your feedback~!

Thank you~

Xintong Song

Reply via email to