[jira] [Created] (FLINK-13832) DefaultRollingPolicy create() method should be renamed to builder()

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Created] (FLINK-13832) DefaultRollingPolicy create() method should be renamed to builder()

Shang Yuanchun (Jira)
Gyula Fora created FLINK-13832:
----------------------------------

             Summary: DefaultRollingPolicy create() method should be renamed to builder()
                 Key: FLINK-13832
                 URL: https://issues.apache.org/jira/browse/FLINK-13832
             Project: Flink
          Issue Type: Improvement
          Components: Connectors / FileSystem
    Affects Versions: 1.9.0
            Reporter: Gyula Fora


The  DefaultRollingPolicy.create() method returns an instance of PolicyBuilder not a DefaultRollingPolicy.  Therefore we should add a new method named .builder() and deprecate "create".

Right now if we want to create a new instance with the default settings we have to call:

DefaultRollingPolicy.create().build()


This nicely illustrates the problem with the naming of this method.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)