Github user StefanRRichter commented on a diff in the pull request: https://github.com/apache/flink/pull/5239#discussion_r168168634 --- Diff: flink-contrib/flink-statebackend-rocksdb/src/main/java/org/apache/flink/contrib/streaming/state/RocksDBStateBackend.java --- @@ -104,10 +107,8 @@ @Nullable private OptionsFactory optionsFactory; - /** True if incremental checkpointing is enabled. - * Null if not yet set, in which case the configuration values will be used. */ - @Nullable - private Boolean enableIncrementalCheckpointing; + /** True if incremental checkpointing is enabled. */ + private TernaryBoolean enableIncrementalCheckpointing; --- End diff -- I think by convention, the `is` should not be part of a boolean field name, only replace the `get` in the getter.
---