I have opened the following PR: https://github.com/apache/storm/pull/3546 regarding the same. Please review the same.
Few pointers where I would need help from the community: 1. I don't have an Apache JIRA account yet. Already raised the request a couple of days ago, it seems to be still pending approval. As a result I don't have a corresponding JIRA ticket available for this feature which can be tagged to the pull request and where such discussions can be taken up. 2. I went through the DEVELOPER.md and it mentions: *"See _Development workflow_ above on how you can create a pull request, for instance."* However, I was not able to find the corresponding Development workflow section. Hence, I have prepared the PR based on what I observed in the previous merged PRs. Let me know if anything needs to be changed or please point me to the right section where these details are already documented. 3. I am also looking to make this feature available in older versions starting from 1.x.x till latest. Let me know what is the procedure for that. AFAIS, the flux core got repackaged between this time frame and perhaps I may have to create a separate branch to take care of that for older versions. 4. What will be required from my side to publish the new jar versions for flux-core (and other impacted artifacts) Regards Bharat Gulati On Fri, Jun 16, 2023 at 1:46 PM 6harat <bharat.gulati.ce...@gmail.com> wrote: > Sure, will look to submit a draft PR this weekend. > > On Thu, 15 Jun 2023, 12:20 Richard Zowalla, <rich...@zowalla.com> wrote: > >> Hey, >> >> I think, that it would be a good addition. >> If you have some spare time, I guess, that a PR would be very welcome. >> >> Gruß >> Richard >> >> >> Am 15. Juni 2023 08:16:14 MESZ schrieb 6harat < >> bharat.gulati.ce...@gmail.com>: >> >>> Hey, >>> >>> A while back the support for declaring WorkerHook in TopologyBuilder was >>> added. Related JIRA: https://issues.apache.org/jira/browse/STORM-126 >>> >>> However, this feature has still not made it to FluxBuilder. Wanted to >>> check if there are plans to bring parity in this regard. >>> >>> Seems like a small change only, happy to submit a PR if the feature is >>> accepted. >>> >>> Regards >>> 6harat >>> >>> -- 6harat solr enthusiast [not affiliated to core dev team]
