apache / apache/accumulo-fluo

Observer configuration in fluo.properties needs way to handle commas in values

Open
#521 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
200
Forks
73
PR merge metrics
No merged PRs in 30d

Description

While trying to run the phrasecount application on an EC2 cluster with multiple zookeepers setup, my attempt to call `fluo init` failed because my fluo.properties file was invalid. This was due to an observer being configured with a zookeeper connection string that had commas in it (due to multiple servers). The commas caused an error to be thrown and stopped initialization.

This script creates the `fluo.properties` file:

https://github.com/fluo-io/phrasecount/blob/master/bin/configure.sh#L39

Contributor guide

Open the contributing guide

Research direction

Start with phrasecount/bin/configure.sh at line 39 and reproduce the failure using a zookeeper connection string containing commas in fluo.properties. Trace how fluo init parses observer configuration and identify the relevant tests or initialization path. Done means a comma-containing observer value is accepted and initialization no longer rejects the properties file.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend, infrastructure
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.