-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Include timeout on flowStats #87
Conversation
…://github.com/assimbly/runtime into feature/issue-86/include-timeout-on-flowstats
flow.put("completed",completedMessages); | ||
flow.put("failed",failedMessages); | ||
flow.put("pending",pendingMessages); | ||
flow.put("completedExchanges",completedMessages); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm trying to avoid the term "exchange". It's a concept of Camel, but not of Assimbly. Exchange is a difficult concept as it has a lifecycle and patterns. I want to simplify it to just speak of "messages". So I deliberately removed Exchange everywhere.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this because Dovetail uses it. I think this concept is too difficult for Dovetail users.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, I can change it.
I need to change this also on Dovetail, frontend and backend.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done 👍🏻
@@ -1654,6 +1659,14 @@ public String getFlowStats(String flowId, boolean fullStats, boolean includeStep | |||
|
|||
} | |||
|
|||
private long getTimeout(CamelContext context) throws MalformedObjectNameException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW Cool to add the timeout. Didn't how to do it. Seems a good addition.
Original issue description
Include timeout on flowStats
closes #86