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

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

Github user blrunner commented on the pull request:

    https://github.com/apache/tajo/pull/464#issuecomment-86422673
  
    Thanks for your contribution and I suggest you to consider following 
opinions.
    
    First, I found that you had wrote your name and current date. But we don't 
write our name and some date at all class files. Thus, you need to remove your 
name and date.
    Second, you need to write description for new added class files.
    
    For the reference, I just reviewed the patch simply. I'll review it 
continuously without above comments.


> Implement a "create external table" and external storage with elasticsearch.
> ----------------------------------------------------------------------------
>
>                 Key: TAJO-1451
>                 URL: https://issues.apache.org/jira/browse/TAJO-1451
>             Project: Tajo
>          Issue Type: Sub-task
>          Components: storage
>            Reporter: Henry Jeong
>            Assignee: Henry Jeong
>         Attachments: TAJO-1451.patch
>
>
> To create a small issue in order to easy reivew.
> First of all, I will create a sub project which is tajo-storage-elasticsearch 
> for tajo external storage.
> As soon as possible I will do pull request.
> Thanks.



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

Reply via email to