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

ASF GitHub Bot commented on TRAFODION-3234:
-------------------------------------------

Github user DaveBirdsall commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1746#discussion_r235207369
  
    --- Diff: core/sql/executor/HiveClient_JNI.cpp ---
    @@ -68,6 +68,27 @@ char* HiveClient_JNI::getErrorText(HVC_RetCode errEnum)
         return (char*)hvcErrorEnumStr[errEnum-HVC_FIRST-1];
     }
     
    
+//////////////////////////////////////////////////////////////////////////////
    +// 
    
+//////////////////////////////////////////////////////////////////////////////
    +HiveClient_JNI* HiveClient_JNI::newInstance(NAHeap *heap, HVC_RetCode 
&retCode)
    +{
    +   QRLogger::log(CAT_SQL_HDFS, LL_DEBUG, "HiveClient_JNI::newInstance() 
called.");
    --- End diff --
    
    Just curious: Why do we use CAT_SQL_HDFS in this QRLogger::log call, but 
elsewhere in this module we use CAT_SQL_HBASE? (Perhaps this use is correct and 
the others are all wrong?)


> Add support for hive partitioned table
> --------------------------------------
>
>                 Key: TRAFODION-3234
>                 URL: https://issues.apache.org/jira/browse/TRAFODION-3234
>             Project: Apache Trafodion
>          Issue Type: Improvement
>          Components: sql-cmp, sql-exe
>    Affects Versions: 2.4
>            Reporter: Selvaganesan Govindarajan
>            Assignee: Selvaganesan Govindarajan
>            Priority: Major
>
> Currently Trafodion doesn't support partitioned hive tables. To start with, 
> added a code to get  full meta data information about the hive table.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to