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

Hudson commented on TAJO-1114:
------------------------------

SUCCESS: Integrated in Tajo-master-build #428 (See 
[https://builds.apache.org/job/Tajo-master-build/428/])
TAJO-1114: Improve ConfVars (SessionVar) to take a validator interface to check 
its input. (Jihun Kang via hyunsik) (hyunsik: rev 
f728413fe36918a9b5e2381af951c05361900099)
* tajo-common/src/main/java/org/apache/tajo/validation/PatternValidator.java
* CHANGES
* tajo-common/src/main/java/org/apache/tajo/validation/BooleanValidator.java
* tajo-common/src/test/java/org/apache/tajo/validation/TestValidators.java
* tajo-common/src/main/java/org/apache/tajo/validation/RangeValidator.java
* tajo-common/src/main/java/org/apache/tajo/util/NumberUtil.java
* tajo-common/src/main/java/org/apache/tajo/validation/PathValidator.java
* tajo-common/src/main/java/org/apache/tajo/ConfigKey.java
* tajo-common/src/main/java/org/apache/tajo/validation/Validator.java
* tajo-common/src/main/java/org/apache/tajo/validation/JavaStringValidator.java
* tajo-common/src/main/java/org/apache/tajo/validation/LengthValidator.java
* tajo-common/src/main/java/org/apache/tajo/conf/TajoConf.java
* 
tajo-common/src/main/java/org/apache/tajo/validation/NetworkAddressValidator.java
* tajo-common/src/main/java/org/apache/tajo/validation/NotNullValidator.java
* tajo-common/src/main/java/org/apache/tajo/validation/AbstractValidator.java
* 
tajo-common/src/main/java/org/apache/tajo/validation/ShellVariableValidator.java
* tajo-common/src/main/java/org/apache/tajo/SessionVars.java
* tajo-common/src/main/java/org/apache/tajo/validation/GroupValidator.java
* tajo-common/src/main/java/org/apache/tajo/validation/ClassValidator.java
* tajo-common/src/main/java/org/apache/tajo/validation/MinValidator.java
* tajo-common/src/main/java/org/apache/tajo/validation/MaxValidator.java
* 
tajo-common/src/main/java/org/apache/tajo/validation/ConstraintViolationException.java
* tajo-common/pom.xml
* tajo-common/src/main/java/org/apache/tajo/validation/ConstraintViolation.java
* tajo-common/src/main/java/org/apache/tajo/validation/Validators.java
* tajo-core/src/main/java/org/apache/tajo/engine/query/QueryContext.java


> 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: Improvement
>            Reporter: Hyunsik Choi
>            Assignee: Jihun Kang
>             Fix For: 0.9.1
>
>
> 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