Re: [PR] Add change to VARCHAR in the upgrade guide [datafusion]

2025-06-01 Thread via GitHub
alamb commented on PR #16216: URL: https://github.com/apache/datafusion/pull/16216#issuecomment-2927523703 Thanks @comphead and @zhuqi-lucas -- 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 s

Re: [PR] Add change to VARCHAR in the upgrade guide [datafusion]

2025-06-01 Thread via GitHub
alamb commented on PR #16216: URL: https://github.com/apache/datafusion/pull/16216#issuecomment-2927523426 Had to merge this so we had at least one commit today 😅 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] Add change to VARCHAR in the upgrade guide [datafusion]

2025-06-01 Thread via GitHub
alamb merged PR #16216: URL: https://github.com/apache/datafusion/pull/16216 -- 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: github-unsubscr...@datafusi

Re: [PR] Add change to VARCHAR in the upgrade guide [datafusion]

2025-06-01 Thread via GitHub
alamb commented on PR #16216: URL: https://github.com/apache/datafusion/pull/16216#issuecomment-2926983181 > I remember back then in Oracle days, there was VARCHAR and VARCHAR2 data types, just thinking aloud if it can be reused like VARCHAR is UTF8 ArrowType, VARCHAR2 is UTF8View Th

Re: [PR] Add change to VARCHAR in the upgrade guide [datafusion]

2025-06-01 Thread via GitHub
zhuqi-lucas commented on PR #16216: URL: https://github.com/apache/datafusion/pull/16216#issuecomment-2926807164 > lgtm thanks @alamb > > I remember back then in Oracle days, there was VARCHAR and VARCHAR2 data types, just thinking aloud if it can be reused like VARCHAR is UTF8 ArrowT

Re: [PR] Add change to VARCHAR in the upgrade guide [datafusion]

2025-05-31 Thread via GitHub
comphead commented on code in PR #16216: URL: https://github.com/apache/datafusion/pull/16216#discussion_r2118165176 ## docs/source/library-user-guide/upgrading.md: ## @@ -21,6 +21,55 @@ ## DataFusion `48.0.0` +### `VARCHAR` SQL type changed to map to `Utf8View` Arrow type

[PR] Add change to VARCHAR in the upgrade guide [datafusion]

2025-05-30 Thread via GitHub
alamb opened a new pull request, #16216: URL: https://github.com/apache/datafusion/pull/16216 ## Which issue does this PR close? - Related to https://github.com/apache/datafusion/issues/15096 - Follow on to https://github.com/apache/datafusion/pull/16142 ## Rationale for

Re: [PR] Add change to VARCHAR in the upgrade guide [datafusion]

2025-05-30 Thread via GitHub
alamb commented on code in PR #16216: URL: https://github.com/apache/datafusion/pull/16216#discussion_r2116071996 ## docs/source/library-user-guide/upgrading.md: ## @@ -21,6 +21,55 @@ ## DataFusion `48.0.0` +### `VARCHAR` SQL type changed to map to `Utf8View` Arrow type + +

Re: [PR] Add change to VARCHAR in the upgrade guide [datafusion]

2025-05-30 Thread via GitHub
zhuqi-lucas commented on code in PR #16216: URL: https://github.com/apache/datafusion/pull/16216#discussion_r2116036299 ## docs/source/library-user-guide/upgrading.md: ## @@ -21,6 +21,55 @@ ## DataFusion `48.0.0` +### `VARCHAR` SQL type changed to map to `Utf8View` Arrow ty