Client properties file should be loaded using commons-configuration
- 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
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