[
https://issues.apache.org/jira/browse/TAJO-1971?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15037317#comment-15037317
]
ASF GitHub Bot commented on TAJO-1971:
--------------------------------------
Github user hyunsik commented on a diff in the pull request:
https://github.com/apache/tajo/pull/890#discussion_r46515630
--- Diff:
tajo-catalog/tajo-catalog-drivers/tajo-hive/src/test/java/org/apache/tajo/catalog/store/TestHiveCatalogStore.java
---
@@ -416,8 +416,8 @@ private void testAddPartition(URI uri, String
tableName, String partitionName) t
List<PartitionKeyProto> partitionKeyList = new ArrayList<>();
String[] partitionNames = partitionName.split("/");
- for(int i = 0; i < partitionNames.length; i++) {
- String[] eachPartitionName = partitionNames[i].split("=");
+ for (String partitionName1 : partitionNames) {
--- End diff --
How about just ``partitionName``?
> Replace 'for' loop with 'foreach'
> ---------------------------------
>
> Key: TAJO-1971
> URL: https://issues.apache.org/jira/browse/TAJO-1971
> Project: Tajo
> Issue Type: Improvement
> Reporter: Dongkyu Hwangbo
> Assignee: Dongkyu Hwangbo
> Priority: Trivial
> Labels: newbie
>
> Reports for loops which iterate over collections or arrays, and can be
> replaced with the foreach iteration syntax, available in Java 5 and newer, to
> get the improvement of readability and so on.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
