Skip to content
This repository has been archived by the owner on Feb 8, 2019. It is now read-only.

Gossip 70 #71

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Gossip 70 #71

wants to merge 2 commits into from

Conversation

ankagrawal
Copy link

Hi,

I have created a basic implementation for GOSSIP-70 (https://issues.apache.org/jira/browse/GOSSIP-70). It covers GOSSIP-94, GOSSIP-95, GOSSIP-96 and GOSSIP-97 over Edward's implementation of GOSSIP-93.

I would love to get review comments from all of you to make this code better and to fix any errors.

Thanks
Ankur

Edward Capriolo and others added 2 commits August 10, 2017 23:10
Basic implementation of GOSSIP-94, GOSSIP-95, GOSSIP-96 and GOSSIP-97
over GOSSIP-93
Copy link
Contributor

@edwardcapriolo edwardcapriolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall change the spacing to two space for indent not tab. I will test out and make other comments.

package org.apache.gossip.consistency;

public enum ConsistencyLevel {
ALL,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indenting is two spaces

if(allowedFailures < 0)
{
cancelAll(futures);
System.out.println(ex.getStackTrace().toString());
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use loggers not printlny

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants