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

Hudson commented on TAJO-1570:
------------------------------

SUCCESS: Integrated in Tajo-master-build #678 (See 
[https://builds.apache.org/job/Tajo-master-build/678/])
TAJO-1570: CatalogUtil newSimpleDataTypeArray should use newSimpleDataType. 
(jihoonson: rev 1bceb0e11ae6626e0aa2a603fab984d16105c849)
* 
tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java
* CHANGES


> CatalogUtil newSimpleDataTypeArray should use newSimpleDataType
> ---------------------------------------------------------------
>
>                 Key: TAJO-1570
>                 URL: https://issues.apache.org/jira/browse/TAJO-1570
>             Project: Tajo
>          Issue Type: Improvement
>            Reporter: DaeMyung Kang
>            Assignee: DaeMyung Kang
>            Priority: Minor
>             Fix For: 0.11.0
>
>
> It is better CatalogUtil's newSimpleDataTypeArray method use 
> newSimpleDataType. Because their purpose is the same. 
> and it is efficient when someone change newSimpleDataType method.
> ```java
>    public static DataType [] newSimpleDataTypeArray(Type... types) {
>      DataType [] dataTypes = new DataType[types.length];
>      for (int i = 0; i < types.length; i++) {
>       dataTypes[i] = DataType.newBuilder().setType(types[i]).build();
>      }
>      return dataTypes;
>    }
> ```



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

Reply via email to