Re: [PR] [HIVE-28585]: Ensure SerDes is case insensitive for data type-'string' to align with HQL and SQL [hive]

Mon, 10 Feb 2025 23:20:59 -0800 


zhangbutao commented on PR #5515:
URL: https://github.com/apache/hive/pull/5515#issuecomment-2649993205

   > @zhangbutao Thanks for checking the PR, sorry for the delay
   > 
   > Here is the query from trino `metaTableColumns: value_0,value_1,value_2 
CREATE TABLE your_table_name ( value_0 STRING, value_1 String, value_2 string 
); insert into your_table_name values ("hello","yes , okay","1");`
   > 
   > Failure: <img alt="image" width="762" 
src="https://private-user-images.githubusercontent.com/149884343/399677670-92527733-0b9c-4a0b-9456-a5fa3fa0620e.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkyNDkwMTIsIm5iZiI6MTczOTI0ODcxMiwicGF0aCI6Ii8xNDk4ODQzNDMvMzk5Njc3NjcwLTkyNTI3NzMzLTBiOWMtNGEwYi05NDU2LWE1ZmEzZmEwNjIwZS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjExJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIxMVQwNDM4MzJaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT00Nzc5MzE1ZmFiZWE1ODk4MjQxNmY2NDQ3MWQ0ZTE5YTU4Y2Q5ZGQ3MDRlMjljOGI0ZTFlZjg2NjYzNWFkOTAyJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.48rRe1jsbM2t-stG2SNmaC3BHogJgMW3VmmZ5n6fGa8";>
   
   @shivjha30 I don't understand the example you provided. Trino also doesn't 
support string type. I think the example is unrelated to this PR.  Could you 
please provide a specific example about this PR?
   
   BTW, Can we fix the issue from the Trino side? I am concerned that fixing on 
the hive side will affect more downstream project.
   Thanks.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to