-
Notifications
You must be signed in to change notification settings - Fork 0
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
Driver timestamp #15
Driver timestamp #15
Conversation
Signed-off-by: Guian Gumpac <guiang@bitquilltech.com>
Signed-off-by: Guian Gumpac <guiang@bitquilltech.com>
@@ -86,7 +86,7 @@ const std::unordered_map< std::string, int > data_name_data_type_map = { | |||
{OPENSEARCH_TYPE_NAME_SCALED_FLOAT, SQL_DOUBLE}, | |||
{OPENSEARCH_TYPE_NAME_KEYWORD, SQL_WVARCHAR}, | |||
{OPENSEARCH_TYPE_NAME_TEXT, SQL_WVARCHAR}, | |||
{OPENSEARCH_TYPE_NAME_DATE, SQL_TYPE_TIMESTAMP}, | |||
{OPENSEARCH_TYPE_NAME_TIMESTAMP, SQL_TYPE_TIMESTAMP}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't you keep the date mapping to timestamp here as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I've reverted it back.
@@ -372,7 +372,7 @@ opensearchtype_attr_to_sqldesctype(const ConnectionClass *conn, OID type, int ty | |||
case SQL_TYPE_DATE: | |||
case SQL_TYPE_TIME: | |||
case SQL_TYPE_TIMESTAMP: | |||
return SQL_DATETIME; | |||
return SQL_TYPE_TIMESTAMP; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure this is correct, I believe SQL_DATETIME is the correct return here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has be reverted in 1817773
Signed-off-by: Guian Gumpac <guiang@bitquilltech.com>
Signed-off-by: Guian Gumpac <guiang@bitquilltech.com>
…pensearch-project#1975) * Dev/sl google java format15 (#341) * Spotless apply on 'datasources/**/*.java', 'core/**/*.java', 'docs/**/*.java', 'doctest/**/*.java', 'relase-notes/**/*.java', 'spark/**/*.java' Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * Added more directories to build.gradle Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * spotless apply for plugin. Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * Adding checkstyle ignorefailures. Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> --------- Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * added missing semi colon. Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * Update build.gradle Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com> Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> * Update build.gradle Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> * Update build.gradle Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> --------- Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>
…pensearch-project#1975) * Dev/sl google java format15 (#341) * Spotless apply on 'datasources/**/*.java', 'core/**/*.java', 'docs/**/*.java', 'doctest/**/*.java', 'relase-notes/**/*.java', 'spark/**/*.java' Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * Added more directories to build.gradle Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * spotless apply for plugin. Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * Adding checkstyle ignorefailures. Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> --------- Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * added missing semi colon. Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * Update build.gradle Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com> Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> * Update build.gradle Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> * Update build.gradle Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> --------- Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>
Description
Issues Resolved
https://github.com/opensearch-project/sql/issues/317
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.