-
Notifications
You must be signed in to change notification settings - Fork 77
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
Fixes #22371: missing report in change only #4675
Fixes #22371: missing report in change only #4675
Conversation
PR updated with a new commit |
@@ -1718,7 +1719,15 @@ object ExecutionBatch extends Loggable { | |||
case x if (x <= cardinality) => | |||
filteredReports.map(_.toMessageStatusReport(policyMode)).toList ++ | |||
/* We need to complete the list of correct with missing, if some are missing */ | |||
(x until cardinality).map(i => MessageStatusReport(ReportType.Missing, s"[Missing report #${i}]")).toList |
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.
ReportType.Missing
in place of noAnswerType
is the culprit here
This PR is not mergeable to upper versions. |
PR rebased |
4ddb8ab
to
490f06b
Compare
OK, merging this PR |
https://issues.rudder.io/issues/22371