Skip to content

Commit 52a0f63

Browse files
committed
fix comments
Change-Id: I48dcb32ff0a07654c3ded2b47c9fcac8254c39f0
1 parent 2d2f287 commit 52a0f63

File tree

3 files changed

+1
-17
lines changed

3 files changed

+1
-17
lines changed

docs/layouts/shortcodes/generated/core_configuration.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1012,7 +1012,7 @@
10121012
<td><h5>tag.period-formatter</h5></td>
10131013
<td style="word-wrap: break-word;">with_dashes</td>
10141014
<td><p>Enum</p></td>
1015-
<td>The date format for tag periods.<br /><br />Possible values:<ul><li>"with_dashes": Dates and hours with dashes, e.g., 'yyyy-MM-dd HH'</li><li>"without_dashes": Dates and hours without dashes, e.g., 'yyyyMMdd HH'</li><li>"with_dashes_without_spaces": Dates and hours with dashes without spaces, e.g., 'yyyy-MM-ddHH'</li><li>"without_dashes_and_spaces": Dates and hours without dashes and spaces, e.g., 'yyyyMMddHH'</li></ul></td>
1015+
<td>The date format for tag periods.<br /><br />Possible values:<ul><li>"with_dashes": Dates and hours with dashes, e.g., 'yyyy-MM-dd HH'</li><li>"without_dashes": Dates and hours without dashes, e.g., 'yyyyMMdd HH'</li><li>"without_dashes_and_spaces": Dates and hours without dashes and spaces, e.g., 'yyyyMMddHH'</li></ul></td>
10161016
</tr>
10171017
<tr>
10181018
<td><h5>target-file-size</h5></td>

paimon-common/src/main/java/org/apache/paimon/CoreOptions.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3000,8 +3000,6 @@ public InlineElement getDescription() {
30003000
public enum TagPeriodFormatter implements DescribedEnum {
30013001
WITH_DASHES("with_dashes", "Dates and hours with dashes, e.g., 'yyyy-MM-dd HH'"),
30023002
WITHOUT_DASHES("without_dashes", "Dates and hours without dashes, e.g., 'yyyyMMdd HH'"),
3003-
WITH_DASHES_WITHOUT_SPACES(
3004-
"with_dashes_without_spaces", "Dates and hours with dashes, e.g., 'yyyy-MM-ddHH'"),
30053003
WITHOUT_DASHES_AND_SPACES(
30063004
"without_dashes_and_spaces", "Dates and hours without dashes, e.g., 'yyyyMMddHH'");
30073005

paimon-core/src/main/java/org/apache/paimon/tag/TagPeriodHandler.java

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -61,17 +61,6 @@ public interface TagPeriodHandler {
6161
.toFormatter()
6262
.withResolverStyle(ResolverStyle.LENIENT);
6363

64-
DateTimeFormatter HOUR_FORMATTER_WITHOUT_SPACES =
65-
new DateTimeFormatterBuilder()
66-
.appendValue(YEAR, 1, 10, SignStyle.NORMAL)
67-
.appendLiteral('-')
68-
.appendValue(MONTH_OF_YEAR, 2, 2, SignStyle.NORMAL)
69-
.appendLiteral('-')
70-
.appendValue(DAY_OF_MONTH, 2, 2, SignStyle.NORMAL)
71-
.appendValue(HOUR_OF_DAY, 2, 2, SignStyle.NORMAL)
72-
.toFormatter()
73-
.withResolverStyle(ResolverStyle.LENIENT);
74-
7564
DateTimeFormatter HOUR_FORMATTER_WITHOUT_DASHES_AND_SPACES =
7665
new DateTimeFormatterBuilder()
7766
.appendValue(YEAR, 1, 10, SignStyle.NORMAL)
@@ -199,8 +188,6 @@ protected DateTimeFormatter formatter() {
199188
return HOUR_FORMATTER;
200189
case WITHOUT_DASHES:
201190
return HOUR_FORMATTER_WITHOUT_DASHES;
202-
case WITH_DASHES_WITHOUT_SPACES:
203-
return HOUR_FORMATTER_WITHOUT_SPACES;
204191
case WITHOUT_DASHES_AND_SPACES:
205192
return HOUR_FORMATTER_WITHOUT_DASHES_AND_SPACES;
206193
default:
@@ -229,7 +216,6 @@ protected Duration onePeriod() {
229216
protected DateTimeFormatter formatter() {
230217
switch (formatter) {
231218
case WITH_DASHES:
232-
case WITH_DASHES_WITHOUT_SPACES:
233219
return DAY_FORMATTER;
234220
case WITHOUT_DASHES:
235221
case WITHOUT_DASHES_AND_SPACES:

0 commit comments

Comments
 (0)