You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tajo.apache.org by "Tajo QA (JIRA)" <ji...@apache.org> on 2014/02/10 16:51:19 UTC

[jira] [Commented] (TAJO-592) Improve HCatalogStore

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

Tajo QA commented on TAJO-592:
------------------------------

{color:red}*-1 overall.*{color}  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12627996/TAJO-592.patch
  against master revision 4179a7c.

    {color:green}+1 @author.{color}  The patch does not contain any @author tags.

    {color:green}+1 tests included.{color}  The patch appears to include 1 new or modified test files.

    {color:green}+1 javac.{color}  The applied patch does not increase the total number of javac compiler warnings.

    {color:green}+1 javadoc.{color}  The applied patch does not increase the total number of javadoc warnings.

    {color:green}+1 checkstyle.{color}  The patch generated 0 code style errors.

    {color:red}-1 findbugs.{color}  The patch appears to introduce 8 new Findbugs (version 1.3.9) warnings.

    {color:green}+1 release audit.{color}  The applied patch does not increase the total number of release audit warnings.

    {color:green}+1 core tests.{color}  The patch passed unit tests in tajo-catalog/tajo-catalog-drivers/tajo-hcatalog.

Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/115//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/115//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-hcatalog.html
Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/115//console

This message is automatically generated.

> Improve HCatalogStore
> ---------------------
>
>                 Key: TAJO-592
>                 URL: https://issues.apache.org/jira/browse/TAJO-592
>             Project: Tajo
>          Issue Type: Improvement
>          Components: catalog
>    Affects Versions: 0.8-incubating
>            Reporter: Jaehwa Jung
>            Assignee: Jaehwa Jung
>             Fix For: 0.8-incubating
>
>         Attachments: TAJO-592.patch
>
>
> If hive table used default row format delimiter, hive doesn't return _field.delim_. As the result, tajo can't scan hive tables which used default row format delimiter. So, in this case, tajo must set _field.delim_ to \001.
> And current HCatalogStore just supports TextFile, it should supports RCFile.
> At last, I found that HCatalogStore doesn't set compression and _field.delim_ when it created tables. So,we should implement these.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)