Team,

NIFI-13762 is a good example but we need to try and remember to have good
commit messages.  The repo as a whole is for nifi itself, minifi (java),
c2, the registry, etc..

While the context can always be assumed to be nifi if committing for
registry or minifi or the c2 protocol the commit message should make that
clear.

The commit that just went into the commit log says

"NIFI-13762 Expose processor metrics as a part of FlowInfo"

My immediate thought was there is no way we should be doing such a thing.
Then I traced the path to find this was added support for a minifi concept
in the c2 protocol.  Please do clarify in the commit messages then what the
change was.

A more helpful commit message then would be

NIFI-13762 Add support for exposing processor metrics in FlowInfo to the C2
protocol and use in MiNiF

Thanks
Joe

Reply via email to