Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated rootpath validation method in zookeeper coordinator #438

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

Commits on Aug 3, 2018

  1. Updated rootpath validation method in zookeeper coordinator

    Validate rootpath with Exists instead of Create to avoid 'zk: not authenticated' error
    
    Fixed bug: mock Exists
    
    Fixed bug: zookeeper exists err
    
    Add zookeeper coordinator test on Exists method
    
    Fixed bug: add zookeeper coordinator test on Exists method
    
    Fixed bug: Exists return err message
    
    Update zookeeper coordinator test - Exists return type
    
    Update zookeeper coordinator test - Exists & Create
    
    Fixed zookeeper coordinator test - Exists
    
    Fixed protocol.go formatting issue: LF will be relaced by CRLF
    
    gofmt fix on changed files
    
    Modify coordinator to increase test coverage
    
    Modify coordinator to fix build error
    
    Modify errExists
    
    Modify order of method calls in zk coordinator test
    
    Bug fix: zk coordinator test last Exists call return &zk.Stat{}
    
    Remove return errExist to improve test coverage
    
    Add test cases to helpers - validation_test
    
    Allow blank hostname in TestValidateHostList
    
    Allow blank hostname in ValidateHostPort
    
    Fix broken zk coordinator and allow space in consumer name
    yumHunter committed Aug 3, 2018
    Configuration menu
    Copy the full SHA
    fa17b1b View commit details
    Browse the repository at this point in the history
  2. Fix comments

    yumHunter committed Aug 3, 2018
    Configuration menu
    Copy the full SHA
    65f0059 View commit details
    Browse the repository at this point in the history
  3. Testing rootpath validation function

    yumHunter committed Aug 3, 2018
    Configuration menu
    Copy the full SHA
    309b3ee View commit details
    Browse the repository at this point in the history