simonbence commented on code in PR #8123:
URL: https://github.com/apache/nifi/pull/8123#discussion_r1419082031
##########
nifi-api/src/main/java/org/apache/nifi/controller/status/ConnectionStatus.java:
##########
@@ -22,6 +22,7 @@
/**
*/
public class ConnectionStatus implements Cloneable {
+ private long createdAtInMs;
Review Comment:
The implementation was the trigger for the change but not the sole reason.
The idea was -based on how we use these- that the state snapshot and the time
of relevance travels together in many times which makes it suspicious that they
should be boundled together. I will not touch until you make your rounds, let's
discuss after that how you feel about the approach.
--
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: issues-unsubscr...@nifi.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
