Thank you Patrick and Norbert. I will work on https://issues.apache.org/jira/browse/ZOOKEEPER-2421.
On Sat, May 26, 2018 at 11:40 AM Patrick Hunt <[email protected]> wrote: > You need to have the contributor role assigned to your jira login. I just > added it, you should be able to assign yourself to a jira now. > > Regards, > > Patrick > > On Fri, May 25, 2018 at 8:03 PM Prasanth Mathialagan < > [email protected]> wrote: > > > Thanks Patrick and Brian. I tried to reproduce the flaky test from > > https://issues.apache.org/jira/browse/ZOOKEEPER-3040 by running the test > > locally in a loop but no luck. I will look at the other issues you guys > > suggested and see if I can fix them. > > > > I have a question. How to assign a issue to me? Does it get automatically > > assigned when a patch becomes available? > > > > On Sat, May 19, 2018 at 5:03 PM, Patrick Hunt <[email protected]> wrote: > > > > > There are a bunch of flaky tests, here's a list if you're looking for > > high > > > impact activities: http://bit.ly/2x5iV6z > > > > > > We also have a dashboard for flaky tests if you'd like to work on items > > > that are happening recently: > > > https://builds.apache.org/view/S-Z/view/ZooKeeper/job/ > > > ZooKeeper-Find-Flaky-Tests/lastSuccessfulBuild/artifact/report.html > > > > > > Thanks, > > > > > > Patrick > > > > > > > > > On Fri, May 18, 2018 at 3:32 PM Brian Nixon <[email protected]> > > > wrote: > > > > > > > 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 < > > > > [email protected]> wrote: > > > > > > > > > Thanks Norbert. This is really helpful!! > > > > > > > > > > On Wed, May 16, 2018 at 4:16 AM, Norbert Kalmar < > > [email protected]> > > > > > 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 < > > > > > > [email protected]> 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 > > > > > > > > > > > > > > > > > > > > > > > > > > > >
