[SUB-TASK][KPIP-2] Complete Flink data type to Hive data type mapping #1704
Assignees
Labels
kind:feature
Feature request
Comments
3 tasks
45 tasks
link3280
added a commit
to link3280/kyuubi
that referenced
this issue
Jan 16, 2022
link3280
added a commit
to link3280/kyuubi
that referenced
this issue
Jan 16, 2022
yuruguo
added a commit
to yuruguo/kyuubi
that referenced
this issue
Mar 30, 2023
pan3793
pushed a commit
that referenced
this issue
Mar 30, 2023
### _Why are the changes needed?_ closed #1770 Support flink `time` type in query operation ### _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 #4635 from yuruguo/support-flink-time-type. Closes #4635 9f9a3e7 [Ruguo Yu] [Kyuubi #1704] Support flink time type in query operation Authored-by: Ruguo Yu <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
pan3793
pushed a commit
that referenced
this issue
Mar 30, 2023
### _Why are the changes needed?_ closed #1770 Support flink `time` type in query operation ### _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 #4635 from yuruguo/support-flink-time-type. Closes #4635 9f9a3e7 [Ruguo Yu] [Kyuubi #1704] Support flink time type in query operation Authored-by: Ruguo Yu <[email protected]> Signed-off-by: Cheng Pan <[email protected]> (cherry picked from commit 1f8f0e6) Signed-off-by: Cheng Pan <[email protected]>
pan3793
pushed a commit
that referenced
this issue
Mar 30, 2023
Followup #1704, support flink `time` type in query operation - [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 #4635 from yuruguo/support-flink-time-type. Closes #4635 9f9a3e7 [Ruguo Yu] [Kyuubi #1704] Support flink time type in query operation Authored-by: Ruguo Yu <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
pan3793
pushed a commit
that referenced
this issue
Mar 30, 2023
Followup #1770, support flink `time` type in query operation - [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 #4635 from yuruguo/support-flink-time-type. Closes #4635 9f9a3e7 [Ruguo Yu] [Kyuubi #1704] Support flink time type in query operation Authored-by: Ruguo Yu <[email protected]> Signed-off-by: Cheng Pan <[email protected]> (cherry picked from commit 1f8f0e6) Signed-off-by: Cheng Pan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Code of Conduct
Search before asking
Describe the feature
Currently, only a subset of Flink data types is supported. Queries that return results with unsupported data types would fail with Thrift TTransportExecption. We should fix this by completing Flink data type to Hive data type mapping.
Motivation
This is a sub-task of KPIP-2 #1322.
Describe the solution
Complete mappings in RowSet.
Additional context
No response
Are you willing to submit PR?
The text was updated successfully, but these errors were encountered: