[
https://issues.apache.org/jira/browse/HIVE-2510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13141342#comment-13141342
]
jirapos...@reviews.apache.org commented on HIVE-2510:
-----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2426/#review2985
-----------------------------------------------------------
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
<https://reviews.apache.org/r/2426/#comment6691>
better to use the new DATABASE_NOT_EXISTS error message as well.
Otherwise it looks good.
- Ning
On 2011-11-01 17:10:08, Robert Surówka wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/2426/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2011-11-01 17:10:08)
bq.
bq.
bq. Review request for Yongqiang He, Ning Zhang and Kevin Wilfong.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. Now when creating table in specific database, if that database doesn't
exist Hive will return an error informing about that, instead of throwing Null
Pointer Exception. Null Pointer happens during semantic analysis, so it seems
reasonable to return an error before the code that would throw null pointer
(Warehouse.getDatabasePath()) executes.
bq.
bq.
bq. This addresses bug HIVE-2510.
bq. https://issues.apache.org/jira/browse/HIVE-2510
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. trunk/ql/src/test/queries/clientpositive/input46.q PRE-CREATION
bq. trunk/ql/src/test/results/clientnegative/create_table_failure1.q.out
PRE-CREATION
bq. trunk/ql/src/test/results/clientnegative/create_table_failure2.q.out
PRE-CREATION
bq. trunk/ql/src/test/results/clientnegative/create_table_failure3.q.out
PRE-CREATION
bq. trunk/ql/src/test/results/clientnegative/create_table_failure4.q.out
PRE-CREATION
bq. trunk/ql/src/test/results/clientpositive/input46.q.out PRE-CREATION
bq. trunk/ql/src/test/queries/clientnegative/create_table_failure3.q
PRE-CREATION
bq. trunk/ql/src/test/queries/clientnegative/create_table_failure4.q
PRE-CREATION
bq. trunk/ql/src/test/queries/clientnegative/create_table_failure1.q
PRE-CREATION
bq. trunk/ql/src/test/queries/clientnegative/create_table_failure2.q
PRE-CREATION
bq. trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
1196082
bq. trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/ErrorMsg.java 1196082
bq.
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
1196082
bq.
bq. Diff: https://reviews.apache.org/r/2426/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. Run some sample queries and it seemed to work.
bq.
bq.
bq. Thanks,
bq.
bq. Robert
bq.
bq.
> Hive throws Null Pointer Exception upon CREATE TABLE <db_name>.<table_name>
> .... if the given <db_name> doesn't exist
> -------------------------------------------------------------------------------------------------------------------------
>
> Key: HIVE-2510
> URL: https://issues.apache.org/jira/browse/HIVE-2510
> Project: Hive
> Issue Type: Bug
> Reporter: Robert Surówka
> Assignee: Robert Surówka
> Priority: Trivial
> Attachments: HIVE-2510.1.patch, HIVE-2510.2.patch, HIVE-2510.3.patch,
> HIVE-2510.4.patch, HIVE-2510.5.patch, HIVE-2510.6.patch
>
>
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
