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.

Sign up for GitHub

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

Jump to bottom

[FEATURE] Support Flink's LocalZonedTimestamp DataType #4745

Closed
3 of 4 tasks
link3280 opened this issue Apr 20, 2023 · 0 comments
Closed
3 of 4 tasks

[FEATURE] Support Flink's LocalZonedTimestamp DataType #4745

link3280 opened this issue Apr 20, 2023 · 0 comments
Labels
kind:feature Feature request priority:major

Comments

@link3280
Copy link
Contributor

Code of Conduct

Search before asking

  • I have searched in the issues and found no similar issues.

Describe the feature

LocalZonedTimestamp is used in the result set of the show job statements, which is crucial for Flink job operations.

Motivation

No response

Describe the solution

No response

Additional context

No response

Are you willing to submit PR?

  • Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
  • No. I cannot submit a PR at this time.
link3280 added a commit to link3280/kyuubi that referenced this issue Apr 22, 2023
@link3280
[KYUUBI apache#4745] Support Flink's LocalZonedTimestamp DataType
5f9976d
link3280 added a commit to link3280/kyuubi that referenced this issue Apr 25, 2023
@link3280
[KYUUBI apache#4745] Improve variable naming
462b39f
link3280 added a commit to link3280/kyuubi that referenced this issue Apr 25, 2023
@link3280
[KYUUBI apache#4745] Pin time zone in tests
0693d1f
link3280 added a commit to link3280/kyuubi that referenced this issue Apr 25, 2023
@link3280
[KYUUBI apache#4745] Replace hive's timestamp format with the kyuubi's
e1e900b
pan3793 pushed a commit that referenced this issue Apr 25, 2023
@link3280 @pan3793
[KYUUBI #4745] Support Flink's LocalZonedTimestamp DataType
9e295f9 
### _Why are the changes needed?_

### _How was this patch tested?_
- [X] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [X] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request

Closes #4751 from link3280/KYUUBI-4745.

Closes #4745

e1e900b [Paul Lin] [KYUUBI #4745] Replace hive's timestamp format with the kyuubi's
0693d1f [Paul Lin] [KYUUBI #4745] Pin time zone in tests
462b39f [Paul Lin] [KYUUBI #4745] Improve variable naming
5f9976d [Paul Lin] [KYUUBI #4745] Support Flink's LocalZonedTimestamp DataType

Authored-by: Paul Lin <paullin3280@gmail.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
(cherry picked from commit 79d6645)
Signed-off-by: Cheng Pan <chengpan@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind:feature Feature request priority:major
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@link3280