leonardBang commented on a change in pull request #17749:
URL: https://github.com/apache/flink/pull/17749#discussion_r754130028



##########
File path: docs/content/docs/connectors/table/filesystem.md
##########
@@ -328,15 +328,30 @@ Time extractors define extracting time from partition 
values.
         <td>String</td>
         <td>The extractor class for implement PartitionTimeExtractor 
interface.</td>
     </tr>
+    <tr>
+        <td><h5>partition.time-extractor.formatter-pattern</h5></td>
+        <td style="word-wrap: break-word;">yyyy-MM-dd&nbsp;HH:mm:ss</td>
+        <td>String</td>
+        <td>When 'partition.time-extractor.kind' is set to 'default', you can 
specify a formatter pattern to get a timestamp from partitions. <br>By default, 
the format is 'yyyy-MM-dd&nbsp;HH:mm:ss'.

Review comment:
       How about the name `timestamp-formatter` which has default value 
'yyyy-MM-dd&nbsp;HH:mm:ss'? And it can combine with 
`partition.time-extractor.timestamp-pattern`, and it also should supports 
multiple partition fields like ```$year$month$day```.
   
   




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to