Skip to content
This repository has been archived by the owner on Jan 13, 2023. It is now read-only.

Unable to use configuration with URI Parts #40

Open
juicegit opened this issue Dec 4, 2015 · 1 comment
Open

Unable to use configuration with URI Parts #40

juicegit opened this issue Dec 4, 2015 · 1 comment

Comments

@juicegit
Copy link

juicegit commented Dec 4, 2015

Currently, if I specify the Rabbit Host Parts via topology config, I get a a NPE when it is trying to configure the connection. It looks up HA Hosts and fails because it is null (not specified). If I use "" for Hosts, it fails again trying to use the value as a host. The Config.getFromMap() needs to check for null before trying .toString().

I fixed this in a pull request.

@jnioche
Copy link
Contributor

jnioche commented May 9, 2017

Same issue. It is currently not possible to configure it via the topology config unless you define rabbitmq.ha.hosts, see [https://github.com/ppat/storm-rabbitmq/blob/master/src/main/java/io/latent/storm/rabbitmq/config/ConnectionConfig.java#L176]

Any reason not to fix it?

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

No branches or pull requests

2 participants