Skip to content
This repository was archived by the owner on Aug 13, 2026. It is now read-only.
This repository was archived by the owner on Aug 13, 2026. It is now read-only.

Superfluous declarations of UnknownHostException [JIRA: CLIENTS-614] #552

Description

@christopherfrieler

The constructors of RiakCluster and RiakNode and the corresponding Builders' build()-methods declare the UnknownHostException. But I don't see a way how this exception can occur. May it be that these declarations are superfluous? This might relate to #519.

Activity

  1. changed the title [-]Superfluous declarations of UnknownHostException[/-] [+]Superfluous declarations of UnknownHostException [JIRA: CLIENTS-614][/+] on Sep 27, 2015
  2. alexmoore commented on Sep 28, 2015

    @alexmoore
    Contributor

    Yeah, that looks like a vestigial piece of code that was originally in the RiakNode constructor and branched it's way out. We only need UnknownHostException on RiakNode's start & buildNodes functions, and RiakCluster's addNode I think. I'll cut those declared ones out of the constructors this week sometime for the next release, unless you'd like to submit a PR for them :)

    Thanks!

  3. alexmoore commented on Sep 28, 2015

    @alexmoore
    Contributor

    Also I think the idea was originally we'd check to see if the host was viable when we first created the node, but we just left that up to start for api design reasons.

  4. christopherfrieler commented on Sep 28, 2015

    @christopherfrieler
    ContributorAuthor

    Added pull request to fix this.

  5. alexmoore commented on Feb 2, 2016

    @alexmoore
    Contributor

    Fixed with the merge of #553

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions