[ 
https://issues.apache.org/jira/browse/HIVE-2090?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13020111#comment-13020111
 ] 

Carl Steinbach commented on HIVE-2090:
--------------------------------------

bq. [exec] .../metastore/if/hive_metastore.thrift:212] optional keyword is 
ignored in argument lists.

Looks like this was added here:

{code}
commit 7816572784c7ddafc2c4350b221469af12d198fc
Author: Mark Slee <mcs...@apache.org>
Date:   Mon Sep 10 22:08:49 2007 +0000

Modify Thrift parser to disallow optional/required keywords in argument lists
    
Reviewed By: dreiss
    
Test Plan: Toss an optional/required in an arg list,
check that it's ignored and compiler spits a warning
    
    
git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@665255 
13f79535-47bb-0310-9956-ffa450edef68
{code}

The comments seem to indicate that Thrift does not actually support optional 
parameters or default parameter values for optional parameters, which implies 
that we're depending on undocumented and unintended behavior that can change at 
any time. Can you please ask Mark or David about this checkin and see what they 
say?

Thanks!



> Add "DROP DATABASE ... FORCE"
> -----------------------------
>
>                 Key: HIVE-2090
>                 URL: https://issues.apache.org/jira/browse/HIVE-2090
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore
>            Reporter: Siying Dong
>            Assignee: Siying Dong
>            Priority: Minor
>         Attachments: HIVE-2090.1.patch, HIVE-2090.2.patch, HIVE-2090.3.patch, 
> HIVE-2090.4.patch, HIVE-2090.5.patch
>
>
> A "DROP DATABASE ... FORCE" will be useful, when we use a database for 
> isolation when doing some tests. Being able to force cleaning up the database 
> will make test cleaning up easier.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to