Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[INLONG-10764][SDK] Transform SQL support temporal functions(Including year, quarter, month, week, dayofyear and dayofmonth) #10766

Merged
merged 5 commits into from
Aug 11, 2024

Conversation

yfsn666
Copy link
Contributor

@yfsn666 yfsn666 commented Aug 8, 2024

Fixes #10764

Motivation

Transform SQL support temporal functions(Including year, quarter, month, week, dayofyear and dayofmonth)

Modifications

Add one temporal function class DateExtractFunction and one class DateParser that implements the class ValueParser. Also, add the corresponding unit test codes

Verifying this change

(Please pick either of the following options)

  • This change is a trivial rework/code cleanup without any test coverage.

  • This change is already covered by existing tests in TestTransformTemporalFunctionsProcessor.java

  • This change added tests and can be verified as follows:

    (example:)

    • Added integration tests for end-to-end deployment with large payloads (10MB)
    • Extended integration test for recovery after broker failure

Documentation

  • Does this pull request introduce a new feature? yes
  • If yes, how is the feature documented? not documented

…g year, quarter, month, week, dayofyear and dayofmonth)
@aloyszhang aloyszhang requested review from luchunliang and vernedeng and removed request for luchunliang August 9, 2024 01:48
aloyszhang
aloyszhang previously approved these changes Aug 9, 2024
luchunliang
luchunliang previously approved these changes Aug 9, 2024
@yfsn666 yfsn666 dismissed stale reviews from luchunliang and aloyszhang via e817ffc August 9, 2024 04:01
luchunliang
luchunliang previously approved these changes Aug 9, 2024
aloyszhang
aloyszhang previously approved these changes Aug 9, 2024
@yfsn666 yfsn666 dismissed stale reviews from aloyszhang and luchunliang via f2283fa August 9, 2024 06:30
@dockerzhang dockerzhang merged commit bb00082 into apache:master Aug 11, 2024
8 checks passed
emptyOVO pushed a commit to emptyOVO/inlong that referenced this pull request Aug 15, 2024
…g year, quarter, month, week, dayofyear and dayofmonth) (apache#10766)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants