apache / apache/accumulo

Client properties file should be loaded using commons-configuration

Open
#5,428 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
1.2k
Forks
487
Avg merge
4d 5h
Merged PRs (30d)
13

Description

The code for loading client props seems to use java Properties and not commons config, so FileHandler would not be applicable there. https://github.com/apache/accumulo/blob/cf42adff3f988ff467ff3f71449732b32703573f/core/src/main/java/org/apache/accumulo/core/clientImpl/ClientInfoImpl.java#L113

_Originally posted by @keith-turner in https://github.com/apache/accumulo/issues/5233#issuecomment-2576448056_

Contributor guide

Open the contributing guide

Research direction

Start in core/src/main/java/org/apache/accumulo/core/clientImpl/ClientInfoImpl.java around line 113, where the client properties are loaded. Compare that loading path with commons-configuration and its FileHandler support; the work is done when client properties use the commons-configuration path described by the issue.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.