Hi Prasanth, If you need somewhere to start, ZOOKEEPER-2421 ( https://issues.apache.org/jira/browse/ZOOKEEPER-2421) might be straightforward. :)
It's also the case that there are a bunch of bug report jira's that would benefit from someone building a reproducer. For example, ZOOKEEPER-2776 ( https://issues.apache.org/jira/browse/ZOOKEEPER-2776) hasn't gotten much attention but that would change if a patch with a unit test to reproduce the behavior were attached to the ticket. -Brian On Wed, May 16, 2018 at 4:44 PM, Prasanth Mathialagan < prasanthmathiala...@gmail.com> wrote: > Thanks Norbert. This is really helpful!! > > On Wed, May 16, 2018 at 4:16 AM, Norbert Kalmar <nkal...@cloudera.com> > wrote: > > > Hi Prasanth, > > > > Great to hear, welcome to the community! :) > > Here is how I started: I forked Apache/ZooKeeper ( > > https://github.com/apache/zookeeper) to my own github repository. You > can > > make the changes on your own fork, for example on master. When you are > > done, you can create a pull request on github, you'll find the button for > > it when you are looking at your branch on the website. > > > > There is quite a few open issues: > > https://issues.apache.org/jira/projects/ZOOKEEPER/issues/ > > You can also create new jira's. For starters, I would recommend some very > > basic change, if you don't find any in the already open jira's, just > create > > one (fixing a test is a good start). > > > > Basically this is all you need to start. > > When you checkout ZooKeeper, you can run "ant eclipse" which will > generate > > a .project file for you, you can also import it in idea. (If you haven't > > already) > > > > Unit tests run for an extended time due to mixing unit tests with > > integration tests (there is a plan to seperate them). So be vary. > > > > Once you create a pull request, we will review it (usually reviews come > > quite quick, but you can ping us here in the mailing list). > > A committer will need to commit your change. This could take more time > due > > to various reasons (for example we are in the middle of a new release > right > > now). > > > > Also, it is probably a good idea to also have the fix for older branches > > like 3.5 and 3.4, but usually 3.5 is compatible with master, and 3.4 is > not > > always required. Just create the PR for master and we will have. > > > > I hope this helps! > > > > Regards, > > Norbert > > > > > > > > On Wed, May 16, 2018 at 5:20 AM Prasanth Mathialagan < > > prasanthmathiala...@gmail.com> wrote: > > > > > Greetings!! > > > > > > I have been working with Zookeeper for a while and I would love to know > > > more and contribute. I got good experience with distributed systems > > through > > > my Masters in Computer Science. I was thinking that I could improve the > > > test cases or add more tests so that I can understand ZK better but I > > don't > > > know where to start. It would be helpful if you guys can provide a > > starting > > > point. I am happy to fix long standing bugs as well. > > > > > > Regards, > > > Prasanth > > > > > >
