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

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

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

    https://github.com/apache/tajo/pull/621#discussion_r34954097
  
    --- Diff: 
tajo-catalog/tajo-catalog-drivers/tajo-hive/src/main/java/org/apache/tajo/catalog/store/HiveCatalogUtil.java
 ---
    @@ -95,7 +99,7 @@ public static String 
getHiveFieldType(TajoDataTypes.DataType dataType) {
         case DATE: return serdeConstants.DATE_TYPE_NAME;
         case TIMESTAMP: return serdeConstants.TIMESTAMP_TYPE_NAME;
         default:
    -      throw new CatalogException(dataType + " is not supported.");
    +      throw ExceptionUtil.makeInvalidDataType(dataType);
    --- End diff --
    
    Please shrink this to just ```makeInvalidDataType()```.


> Refactor client errors and exceptions
> -------------------------------------
>
>                 Key: TAJO-1670
>                 URL: https://issues.apache.org/jira/browse/TAJO-1670
>             Project: Tajo
>          Issue Type: Sub-task
>          Components: Java Client, TajoMaster, Worker
>            Reporter: Hyunsik Choi
>            Assignee: Hyunsik Choi
>             Fix For: 0.11.0
>
>
> The main objective is to refactor the error and exception system in client 
> side and its connected parts like TajoMaster.



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

Reply via email to