NIFI-13544 Align number of executors with concurrent tasks in PublishGCPubSub#9454
NIFI-13544 Align number of executors with concurrent tasks in PublishGCPubSub#9454exceptionfactory merged 1 commit intoapache:mainfrom
Conversation
greyp9
left a comment
greyp9
left a comment
There was a problem hiding this comment.
Nice fix @pvillard31.
Having a good test setup to exercise this processor (from working NIFI-13927), I gave it a try.
Interestingly, the processor default of 1 concurrent task also solved the hang observed in that JIRA, where using the former default of 5 and the old processor code caused the hang to happen.
exceptionfactory
left a comment
exceptionfactory
left a comment
There was a problem hiding this comment.
Thanks for implementing this proposed solution @pvillard31, and thanks for verifying the behavior @greyp9, I agree this looks good. +1 merging
Thanks again to @PACordonnier for raising the initial issue!
|
Hello @pvillard31, is it possible to report this fix on 1.28.1 version please? |
|
@fpierrick - NiFi 1.28.1 has already been released and there is no plan for a new 1.x release unless critical security fixes need to be delivered on the 1.x release line (see https://nifi.apache.org/download/). Your best options are either going to NiFi 2, or applying this change to the source code of 1.28.1 and build NiFi yourself. |
…GCPubSub (apache#9454) Co-authored-by: Paul-Adrien Cordonnier <pauladrien@adaltas.com> Signed-off-by: David Handermann <exceptionfactory@apache.org>
5 participants
Summary
NIFI-13544 - Align number of executors with concurrent tasks in PublishGCPubSub
See #9405