Skip to content

Conversation

yx-zhang
Copy link
Contributor

@yx-zhang yx-zhang commented Mar 1, 2024

.collect(Collectors.toList());
TsBlockBuilder tsBlockBuilder = new TsBlockBuilder(outputDataTypes);
tsBlockBuilder.getTimeColumnBuilder().writeLong(0L);
tsBlockBuilder.getColumnBuilder(0).writeLong(System.currentTimeMillis());
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any issue when the timestamp_precision is ns or us? @JackieTien97

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a long value, how to resolve this long value depends on the user. We can default return ms, in stardand sql, there will be a parameter called precision in current_timestamp function to decide whether to return ms, us or ns, so I think it's ok to defaultly return ms here and this function call's time precision should have nothing to do with our timestamp_precision configuration.

@yx-zhang yx-zhang force-pushed the show_current_timestamp branch from fa61eaa to a486fb5 Compare March 1, 2024 06:29
Copy link
Member

@Beyyes Beyyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yx-zhang yx-zhang force-pushed the show_current_timestamp branch from 6a50fac to fad6e18 Compare March 1, 2024 14:04
@qiaojialin qiaojialin merged commit 2b066d8 into apache:master Mar 2, 2024
SzyWilliam pushed a commit to SzyWilliam/iotdb that referenced this pull request Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants