-
Notifications
You must be signed in to change notification settings - Fork 76
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
Add log message when pipe is skipped #6550
Conversation
@@ -103,6 +103,7 @@ protected PipeRunResult processPipe(PipeLine pipeLine, IPipe pipe, Message messa | |||
if (pipe instanceof FixedForwardPipe) { | |||
FixedForwardPipe ffPipe = (FixedForwardPipe) pipe; | |||
if (ffPipe.skipPipe(message, pipeLineSession)) { | |||
log.info("skipping pipe [{}] because of empty input", pipe::getName); |
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.
Het log statement op deze plek klopt niet helemaal met de controle dat uitgevoerd wordt in de skipPipe
methode..
De skip on empty & is not empty check is slechts één van de meerdere redenen waarom deze methode true
zou kunnen retourneren..
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.
Klopt ik zag het ook. Ben er nog niet helemaal over uit of ik een nieuwe processor wil maken of überhaupt de code wil verplaatsen.
Het zit nu iig best raar in elkaar. Voor de 7.9 en 8.0 moeten we denk ik de log regel verplaatsen naar daar waar de checks worden gedaan.
@@ -385,7 +385,7 @@ | |||
<dependency> | |||
<groupId>com.github.mwiede</groupId> | |||
<artifactId>jsch</artifactId> | |||
<version>0.2.14</version> | |||
<version>0.2.15</version> |
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.
Oh, durf je dit hier toch wel aan? Prima op zich. Had het teruggedraaid in mijn PRs.
No description provided.