Github user azagrebin commented on a diff in the pull request: https://github.com/apache/flink/pull/6277#discussion_r200950760 --- Diff: flink-core/src/main/java/org/apache/flink/api/common/state/StateTtlConfiguration.java --- @@ -93,4 +101,59 @@ public Time getTtl() { public TtlTimeCharacteristic getTimeCharacteristic() { return timeCharacteristic; } + + @Override + public String toString() { + return "StateTtlConfiguration{" + + "ttlUpdateType=" + ttlUpdateType + + ", stateVisibility=" + stateVisibility + + ", timeCharacteristic=" + timeCharacteristic + + ", ttl=" + ttl + + '}'; + } + + public static Builder newBuilder() { --- End diff -- I think when we create the new builder the configuration of at least TTL time should be enforced already here as a parameter because the default value for it is not obvious and user should be enforced to think about it.
---