-
Notifications
You must be signed in to change notification settings - Fork 3
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
DROP DATABASE
generates a postgres issue after 1.0 TDP release
#7
Comments
After checking if new 1.0 build could be in cause, I just see versioning commit on top of 0.1.0-SNAPSHOT, nothing that could justify such regression. Have you confirmed that the bug disappears with 0.1.0-SNAPSHOT ? |
yes |
The problem has been identified. Following the merge of TOSIT-IO/hive-old#5 we (DGFiP) forgot to build Hive, so we never tested this change before today. |
thank you @rpignolet 👍 |
We (DGFiP) believe that it is TOSIT-IO/hive-old#5 that introduces this It is not blocking for the TDP 1.0 release. |
DROP DATABASE
generate a postgres issue after 1.0 TDP release DROP DATABASE
generates a postgres issue after 1.0 TDP release
Hello everyone, I confirm that the bug disappears with 0.1.0-SNAPSHOT. Thanks, |
I confirm this error with I think the PR TOSIT-IO/hive-old#5 added the fix https://issues.apache.org/jira/browse/HIVE-22546, which renames the following tables' names (including columns & index) in uppercase and is valid for HIVE v4.0.0:
With this error, it seems that the HIVE v3.1.0 JAVA code is still using these tables' names in lowercase. We could remove the merge of the PR TOSIT-IO/hive-old#5 to avoid this error. But if "DataNucleus Core" is used, the error on the table "RUNTIME_STATS" could occure according to the issue https://issues.apache.org/jira/browse/HIVE-22101. Because "DataNucleus" is part of HIVE package:
|
I confirm the following WARNING about the issue https://issues.apache.org/jira/browse/HIVE-22101 in HIVE
|
To test, from beeline :
CREATE DATABASE test LOCATION test
then
DROP DATABASE test
Table will be well dropped with an error:
The text was updated successfully, but these errors were encountered: