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

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

Github user hyunsik commented on the pull request:

    https://github.com/apache/tajo/pull/206#issuecomment-60461282
  
    Even through it needs rebase, it was easy to merge the patch against the 
latest revision.
    
    I'm reviewing this patch. I'll leave comments soon.


> Improve ConfVars (SessionVar) to take a validator interface to check its 
> input.
> -------------------------------------------------------------------------------
>
>                 Key: TAJO-1114
>                 URL: https://issues.apache.org/jira/browse/TAJO-1114
>             Project: Tajo
>          Issue Type: Task
>            Reporter: Hyunsik Choi
>            Assignee: Jihun Kang
>
> Each task uses session variables from client or configs from tajo-site.xml. 
> The session variables and configs are used in task during query processing.
> If invalid configs or invalid session variables are used for a query, the 
> error will be caused in a runtime instead of query init phase. It's bad 
> situation. Invalid values must be detected at the initialization phase.
>  Because each session variable or config varies in its value type and value 
> range, It would be great if each ConfigKey takes a validator for its value 
> and Tajo uses validators for it.



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

Reply via email to