Skip to content

Document Java System Properties - #6534

Merged
dlmarion merged 6 commits into
apache:mainfrom
Amemeda:4935-system-props
Sep 17, 2026
Merged

dlmarion merged 6 commits into
apache:mainfrom
Amemeda:4935-system-props

Conversation

@Amemeda

@Amemeda Amemeda commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Documenting Java System Properties in ConfigurationDocGen.java:

  • New section for Java System Properties added into the server properties documentation
  • New method Property.getSystemProperties() was added for use by ConfigurationDocGen.java.
  • The compression algorithm System Properties classes were updated to move the property name into a constant to be grabbed easier by Property.getSystemProperties().

The following properties were identified to be System Properties, and are included in this new documentation:

  • CODEC_BZIP2: io.compression.codec.bzip2.class (new constant)
  • CODEC_LZ4: io.compression.codec.lz4.class (new constant)
  • CODEC_LZO io.compression.codec.lzo.class (new constant)
  • CODEC_SNAPPY: io.compression.codec.snappy.class (new constant)
  • CODEC_ZSTANDARD: io.compression.codec.zstd.class (new constant)
  • ACCUMULO_PROPERTIES_PROPERTY: accumulo.properties
  • DEBUG_HOST: org.apache.accumulo.client.rpc.debug.host

Auto-generated server-properties.md with the new System Properties section:

image

Local version of accumulo website with new System Properties section:

image

This pr closes #4935

@Amemeda

Amemeda commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

@dlmarion Could I get your opinion on my output/format of the System Properties? The section is at the very bottom of server-properties.md, proper descriptions still need to be added

image

Comment thread core/src/main/java/org/apache/accumulo/core/conf/Property.java Outdated
Comment thread core/src/main/java/org/apache/accumulo/core/conf/Property.java Outdated
Comment thread core/src/main/java/org/apache/accumulo/core/conf/Property.java Outdated
Comment thread core/src/main/java/org/apache/accumulo/core/conf/ConfigurationDocGen.java Outdated
@Amemeda
Amemeda marked this pull request as ready for review September 11, 2026 19:34
@Amemeda
Amemeda requested a review from dlmarion September 16, 2026 14:31
Comment thread core/src/main/java/org/apache/accumulo/core/conf/Property.java Outdated
@dlmarion
dlmarion merged commit 087855c into apache:main Sep 17, 2026
8 checks passed
@dlmarion dlmarion added this to the 4.0.0 milestone Sep 17, 2026
@Amemeda
Amemeda deleted the 4935-system-props branch September 17, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Java System Properties not documented

2 participants