Skip to content

feat: 新增last和first的column功能实现#102

Merged
zhou-hao merged 4 commits intohs-web:4.2from
goten7:feat-4.2-func-1
Apr 10, 2026
Merged

feat: 新增last和first的column功能实现#102
zhou-hao merged 4 commits intohs-web:4.2from
goten7:feat-4.2-func-1

Conversation

@goten7
Copy link
Copy Markdown
Contributor

@goten7 goten7 commented Apr 9, 2026

No description provided.

@goten7
Copy link
Copy Markdown
Contributor Author

goten7 commented Apr 9, 2026

@zhou-hao

String timeColumn = (String) opts.get(TIME_COLUMN);
fullTimeColumn = metadata.getDialect().buildColumnFullName(table, timeColumn);
} else {
fullTimeColumn = metadata.getDialect().buildColumnFullName(table, "timestamp");
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

从table里找时间类型字段 更合适?


private final SqlFragments FUNCTION;

private static final List<SQLType> timeTypes = Lists.newArrayList(JDBCType.TIMESTAMP, JDBCType.DATE, JDBCType.TIME);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

TIME_WITH_TIMEZONE,TIMESTAMP_WITH_TIMEZONE 应该也支持

}
}
}
if (fullTimeColumn == null) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

这种情况应该直接抛出异常?

@zhou-hao
Copy link
Copy Markdown
Member

zhou-hao commented Apr 9, 2026

单元测试

@goten7 goten7 requested a review from zhou-hao April 9, 2026 12:14
@zhou-hao zhou-hao merged commit 000d8d8 into hs-web:4.2 Apr 10, 2026
1 check failed
@goten7 goten7 deleted the feat-4.2-func-1 branch April 10, 2026 02:29
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.

2 participants