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

ASF GitHub Bot commented on TAJO-1122:
--------------------------------------

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

    https://github.com/apache/tajo/pull/247#discussion_r20848470
  
    --- Diff: 
tajo-storage/tajo-storage-common/src/main/java/org/apache/tajo/storage/StorageManager.java
 ---
    @@ -22,28 +22,35 @@
     import org.apache.commons.logging.Log;
     import org.apache.commons.logging.LogFactory;
     import org.apache.hadoop.conf.Configuration;
    -import org.apache.hadoop.fs.*;
    +import org.apache.hadoop.fs.FileStatus;
    +import org.apache.hadoop.fs.FileSystem;
    +import org.apache.hadoop.fs.Path;
     import org.apache.tajo.*;
    -import org.apache.tajo.catalog.*;
    +import org.apache.tajo.catalog.Schema;
    +import org.apache.tajo.catalog.SortSpec;
    +import org.apache.tajo.catalog.TableDesc;
    +import org.apache.tajo.catalog.TableMeta;
     import org.apache.tajo.catalog.proto.CatalogProtos;
     import org.apache.tajo.catalog.proto.CatalogProtos.FragmentProto;
     import org.apache.tajo.catalog.proto.CatalogProtos.StoreType;
     import org.apache.tajo.conf.TajoConf;
     import org.apache.tajo.conf.TajoConf.ConfVars;
     import org.apache.tajo.plan.LogicalPlan;
    -import org.apache.tajo.plan.logical.*;
    +import org.apache.tajo.plan.logical.LogicalNode;
    +import org.apache.tajo.plan.logical.NodeType;
    +import org.apache.tajo.plan.logical.ScanNode;
     import org.apache.tajo.plan.rewrite.RewriteRule;
    -import org.apache.tajo.storage.fragment.FileFragment;
     import org.apache.tajo.storage.fragment.Fragment;
     import org.apache.tajo.storage.fragment.FragmentConvertor;
    -import org.apache.tajo.storage.hbase.HBaseStorageManager;
     import org.apache.tajo.util.TUtil;
     
     import java.io.IOException;
     import java.lang.reflect.Constructor;
     import java.net.URI;
     import java.text.NumberFormat;
    -import java.util.*;
    +import java.util.HashMap;
    --- End diff --
    
    It is not unused.


> Refactor the tajo-storage project structure.
> --------------------------------------------
>
>                 Key: TAJO-1122
>                 URL: https://issues.apache.org/jira/browse/TAJO-1122
>             Project: Tajo
>          Issue Type: Sub-task
>            Reporter: Hyoungjun Kim
>            Assignee: Hyoungjun Kim
>            Priority: Minor
>
> 'tajo-storage' sub-project will be changed to the following structure which 
> is more flexible for adding new storage.
> {noformat}
> tajo-storage
>   tajo-storage-common
>     src
>     pom.xml
>   tajo-external-storage
>     tajo-hbase
>       src
>       pom.xml
>     tajo-<other storage>
>       src
>       pom.xml
>     pom.xml
>   pom.xml
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to