gianm commented on issue #12643: URL: https://github.com/apache/druid/issues/12643#issuecomment-1155989257
The test case was added in #11419, which was about improving error message. The case is a little strange: it uses a mocked work item whose ID changes across calls to `getTaskId`. No real work item does that. There's a comment `// modify taskId to make task assignment failed` that suggests this is being done to force the RemoteTaskRunner down an assignment-failed path, so we can verify that the error message comes out as expected. We may want to rethink how the test case works, so it can be more realistic. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
