Github user azagrebin commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6277#discussion_r200974407
  
    --- 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() {
    +           return new Builder();
    +   }
    +
    +   /**
    +    * Builder for the {@link StateTtlConfiguration}.
    +    */
    +   public static class Builder {
    +
    +           private TtlUpdateType ttlUpdateType;
    --- End diff --
    
    I would initialise these fields with some reasonable default values, like 
`OnCreateAndWrite`, `NeverReturnExpired`, and `ProcessingTime`.


---

Reply via email to