You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@hive.apache.org by GitBox <gi...@apache.org> on 2021/02/11 20:26:56 UTC
[GitHub] [hive] nrg4878 opened a new pull request #1972: HIVE-24770: Change classname for MultiDelimiterSerDe during upgrade (…
nrg4878 opened a new pull request #1972:
URL: https://github.com/apache/hive/pull/1972
…Naveen Gangam)
HIVE-20619 makes MultiDelimiterSerDe a first class serde in hive moving it from hive-contrib into hive-exec jar. Part of the change includes removing "contrib" from its full qualified package+class name. Although, this does not introduce a backward incompatibility, it still requires users to manually add hive-contrib.jar to the classpath to make it work after the upgrade. (This is no different than what a user would do after upgrade without this change as well). However, we can make the user experience better by automatically changing the hive table metadata to rename the class.
### What changes were proposed in this pull request?
Schema upgrade scripts update the SLIB column value in SERDES table to update it from "org.apache.hadoop.hive.contrib.serde2.MultiDelimitSerDe" to "org.apache.hadoop.hive.serde2.MultiDelimitSerDe"
### Why are the changes needed?
Better user experience to getting MultiDelimiterSerDe to work out of the box after the upgrade.
### Does this PR introduce _any_ user-facing change?
NO
### How was this patch tested?
Manually.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] nrg4878 commented on pull request #1972: HIVE-24770: Change classname for MultiDelimiterSerDe during upgrade (…
Posted by GitBox <gi...@apache.org>.
nrg4878 commented on pull request #1972:
URL: https://github.com/apache/hive/pull/1972#issuecomment-783517711
Thanks for the review @yongzhi. Fix has been committed to master. Closing the PR
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] nrg4878 closed pull request #1972: HIVE-24770: Change classname for MultiDelimiterSerDe during upgrade (…
Posted by GitBox <gi...@apache.org>.
nrg4878 closed pull request #1972:
URL: https://github.com/apache/hive/pull/1972
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org