ykisana commented on issue #20435:
URL: https://github.com/apache/druid/issues/20435#issuecomment-6092403280

   > 6\. [@rishi-rana](https://github.com/rishi-rana) I think this fixes the 
test. But I believe there may be an underlying bug here.
   >         The Overlord's HttpRemoteTaskRunner.streamTaskLog passes the 
worker's response through without checking the HTTP status. A 404, which 
happens once the worker has dropped a finished task, becomes a 
present-but-empty log stream.
   >         SwitchingTaskLogStreamer then never falls back to deep storage, 
where the log actually is.
   >         
https://github.com/apache/druid/blob/98bbcb4c7f78/indexing-service/src/main/java/org/apache/druid/indexing/overlord/hrtr/HttpRemoteTaskRunner.java#L936-L940
   
   Nvm ignore me, looking at logs, the test flake happened before this bug 
would've opened up


-- 
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]

Reply via email to