Skip to content

Commit 768032e

Browse files
branch-4.0: [opt](catalog) make catalog createtime the same format as updatetime #59288 (#59306)
Cherry-picked from #59288 Co-authored-by: camby <[email protected]>
1 parent 97ea173 commit 768032e

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

fe/fe-core/src/main/java/org/apache/doris/catalog/JdbcResource.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
import org.apache.doris.common.plugin.CloudPluginDownloader;
2727
import org.apache.doris.common.plugin.CloudPluginDownloader.PluginType;
2828
import org.apache.doris.common.proc.BaseProcResult;
29+
import org.apache.doris.common.util.TimeUtils;
2930
import org.apache.doris.common.util.Util;
3031
import org.apache.doris.datasource.CatalogProperty;
3132
import org.apache.doris.datasource.ExternalCatalog;
@@ -47,8 +48,6 @@
4748
import java.net.URISyntaxException;
4849
import java.security.MessageDigest;
4950
import java.security.NoSuchAlgorithmException;
50-
import java.time.LocalDateTime;
51-
import java.time.ZoneId;
5251
import java.util.ArrayList;
5352
import java.util.Arrays;
5453
import java.util.List;
@@ -206,7 +205,7 @@ protected void setProperties(ImmutableMap<String, String> properties) throws Ddl
206205
this.configs = Maps.newHashMap(properties);
207206
validateProperties(this.configs);
208207
applyDefaultProperties();
209-
String currentDateTime = LocalDateTime.now(ZoneId.systemDefault()).toString().replace("T", " ");
208+
String currentDateTime = TimeUtils.longToTimeString(System.currentTimeMillis());
210209
configs.put(CREATE_TIME, currentDateTime);
211210
// check properties
212211
for (String property : ALL_PROPERTIES) {

fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/CreateCatalogCommand.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import org.apache.doris.common.ErrorCode;
2424
import org.apache.doris.common.ErrorReport;
2525
import org.apache.doris.common.util.PropertyAnalyzer;
26+
import org.apache.doris.common.util.TimeUtils;
2627
import org.apache.doris.common.util.Util;
2728
import org.apache.doris.datasource.ExternalCatalog;
2829
import org.apache.doris.datasource.InternalCatalog;
@@ -35,8 +36,6 @@
3536
import com.google.common.base.Strings;
3637
import com.google.common.collect.Maps;
3738

38-
import java.time.LocalDateTime;
39-
import java.time.ZoneId;
4039
import java.util.Map;
4140
import java.util.Objects;
4241

@@ -81,7 +80,7 @@ private void validate(ConnectContext ctx) throws AnalysisException {
8180
+ " to enable it temporarily.");
8281
}
8382

84-
String currentDateTime = LocalDateTime.now(ZoneId.systemDefault()).toString().replace("T", " ");
83+
String currentDateTime = TimeUtils.longToTimeString(System.currentTimeMillis());
8584
properties.put(ExternalCatalog.CREATE_TIME, currentDateTime);
8685
PropertyAnalyzer.checkCatalogProperties(properties, false);
8786
}

0 commit comments

Comments
 (0)