jkrauss82 commented on PR #9153: URL: https://github.com/apache/storm/pull/9153#issuecomment-6439817829
Hi @GGraziadei and @rzo1 and thanks for your patience as well and you detailed comments. I have made the adaptions through my local AI and pushed the changes, also I have updated to description of the PR to reflect the adapted scope. I hope this is how you intended it to be changed. On a side note: we had a couple of cluster restarts which were done with the custom storm build and the supervisors had no crashes anymore. Interestingly this surfaced a crash in nimbus due to a race condition when looking for an artifact and downloading that (I think there is even a comment in the code at some location about this race). This seems like something which should be handled in another related PR as well, I will submit an issue for this separately. -- 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]
