Hi, Bhargava, thanks for your help!
you should be able to raise a PR I believe. 1. clone the repo to your own space. 2. create your own feature branch and commit the changes. 3. git push to your repo There should be a message in github heron repo about creating a new PR for the change. Please feel free to ping me if you have any questions. On Mon, Dec 17, 2018 at 10:39 AM Bhargava Vadlamani <[email protected]> wrote: > Hello , > > I finished updating the tutorial but I guess I cannot raise a PR into the > repo. > > Here are the changes I made: > > 1) Updated doc to point to latest Apache heron doc > > 2)Updated the config (pom.xml) to point to latest stable version of Apache > Heron > > 3) The old code wouldn't even compile since the method signatures and > actual args we pass are two different classes (like storm topology and > heron topology) > > I believe to the best of my knowledge that the code is good to be merged so > that new heron`ers` would take advantage of the tutorial and contribute if > they are interested > > Next steps: > > I am writing some out of the box spouts to read data from Databases so that > users can plugin heron with a database easily > > Thank you @Sree for directing me on this > > Bhargava > > On Mon, Dec 17, 2018, 10:29 AM Ning Wang <[email protected] wrote: > > > Hi All, > > > > It has been two weeks since our last sync and it is time to share our > > progress again. Let's share our works for the last two weeks in this > > thread. And see if we need some hangout discussions or not. > > > > My update: > > - Studied the two existing reduce functions. Before implementing new > > aggregation functions, I am refactoring/cleaning up the existing code and > > have prepared three PRs( > > https://github.com/apache/incubator-heron/pull/3123, > > https://github.com/apache/incubator-heron/pull/3124, > > https://github.com/apache/incubator-heron/pull/3125). > > >
