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

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

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

    https://github.com/apache/tajo/pull/489#discussion_r27721416
  
    --- Diff: 
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/FileAppender.java
 ---
    @@ -53,6 +53,10 @@ public FileAppender(Configuration conf, TaskAttemptId 
taskAttemptId, Schema sche
     
         try {
           if (taskAttemptId != null) {
    +        if (!(conf instanceof TajoConf)) {
    +          throw new IllegalArgumentException("Configuration must be a 
TajoConf instance");
    --- End diff --
    
    How about change the exception message as follows.
    ```
    Configuration must be an instance of TajoConf.
    (or Configuration is not an instance of TajoConf.)
    ```
    



> Add SeekableScanner support to DelimitedTextFileScanner
> -------------------------------------------------------
>
>                 Key: TAJO-1494
>                 URL: https://issues.apache.org/jira/browse/TAJO-1494
>             Project: Tajo
>          Issue Type: New Feature
>          Components: storage
>            Reporter: Jinho Kim
>            Assignee: Jinho Kim
>         Attachments: TAJO-1494.patch, TAJO-1494_2.patch
>
>
> CSVFile will be deprecated. So DelimitedTextFileScanner should support 
> SeekableScanner interface for BSTIndex



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

Reply via email to