[GitHub] [storm] 6harat commented on pull request #3546: [STORM-XXXX] Support for declaring WorkerHook in Flux topology definitions

2023-06-18 Thread via GitHub
6harat commented on PR #3546: URL: https://github.com/apache/storm/pull/3546#issuecomment-1596224842 I was going through other parts of the code and found this TODO. Seems like it was intended to be connected to WorkerHooks when those were first introduced. ``` private Map make

[GitHub] [storm] 6harat commented on pull request #3546: [STORM-XXXX] Support for declaring WorkerHook in Flux topology definitions

2023-06-17 Thread via GitHub
6harat commented on PR #3546: URL: https://github.com/apache/storm/pull/3546#issuecomment-1595677853 Please note the following observations / callouts: 1. have not added workerHoosk to executioncontext. AFAIS constructed spouts and bolts are being stored here so as to allow creation o