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

Hudson commented on TAJO-1971:
------------------------------

FAILURE: Integrated in Tajo-master-CODEGEN-build #652 (See 
[https://builds.apache.org/job/Tajo-master-CODEGEN-build/652/])
TAJO-1971: Replace 'for' loop with 'foreach'. (jihoonson: rev 
0d434cdba4eb08bd1a0f394866b40345bec880b5)
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckClassAdapter.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/ConstantPool.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/AnnotationConstantsCollector.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/AnnotationWriter.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/DistinctGroupbyHashAggregationExec.java
* 
tajo-core-tests/src/test/java/org/apache/tajo/engine/planner/physical/TestHashPartitioner.java
* tajo-core/src/main/java/org/apache/tajo/querymaster/Repartitioner.java
* 
tajo-core-tests/src/test/java/org/apache/tajo/engine/eval/TestEvalTreeUtil.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXClassAdapter.java
* 
tajo-catalog/tajo-catalog-server/src/test/java/org/apache/tajo/catalog/CatalogTestingUtil.java
* tajo-plan/src/main/java/org/apache/tajo/plan/logical/DistinctGroupbyNode.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/AnnotationNode.java
* 
tajo-storage/tajo-storage-common/src/main/java/org/apache/tajo/storage/BaseTupleComparator.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Constant.java
* 
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/rcfile/RCFile.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/ClassWriter.java
* tajo-core/src/main/java/org/apache/tajo/querymaster/Query.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/ASMifier.java
* 
tajo-catalog/tajo-catalog-drivers/tajo-hive/src/main/java/org/apache/tajo/catalog/store/HiveCatalogStoreClientPool.java
* tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/Frame.java
* tajo-common/src/test/java/org/apache/tajo/tuple/memory/TestMemoryRowBlock.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/commons/SerialVersionUIDAdder.java
* 
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/BasicVerifier.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/DistinctGroupbyFirstAggregationExec.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXAnnotationAdapter.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/commons/GeneratorAdapter.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/ASMContentHandler.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/ClassConstantsCollector.java
* 
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/parquet/TajoSchemaConverter.java
* tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/Type.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/commons/Remapper.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/commons/AnalyzerAdapter.java
* tajo-core/src/main/java/org/apache/tajo/master/exec/QueryExecutor.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/DistinctGroupbySortAggregationExec.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/LookupSwitchInsnNode.java
* 
tajo-storage/tajo-storage-hbase/src/main/java/org/apache/tajo/storage/hbase/HBaseTablespace.java
* tajo-jdbc/src/test/java/org/apache/tajo/jdbc/TestTajoJdbc.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/ColPartitionStoreExec.java
* 
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/thirdparty/orc/OrcUtils.java
* CHANGES
* 
tajo-catalog/tajo-catalog-server/src/main/java/org/apache/tajo/catalog/dictionary/InfoSchemaMetadataDictionary.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/commons/LocalVariablesSorter.java
* tajo-core/src/main/java/org/apache/tajo/querymaster/Task.java
* 
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/FileTablespace.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Printer.java
* tajo-plan/src/main/java/org/apache/tajo/plan/serder/EvalNodeSerializer.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/commons/AdviceAdapter.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/Analyzer.java
* tajo-core-tests/src/test/java/org/apache/tajo/master/TestRepartitioner.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/DistinctGroupbySecondAggregationExec.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Shrinker.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/WindowAggExec.java
* tajo-core/src/main/java/org/apache/tajo/master/exec/DDLExecutor.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXCodeAdapter.java
* tajo-core/src/main/java/org/apache/tajo/engine/query/TaskRequestImpl.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/commons/JSRInlinerAdapter.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/MethodWriter.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckMethodAdapter.java
* 
tajo-catalog/tajo-catalog-drivers/tajo-hive/src/test/java/org/apache/tajo/catalog/store/TestHiveCatalogStore.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/SimpleVerifier.java
* tajo-common/src/main/java/org/apache/tajo/util/BytesUtils.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/PhysicalPlannerImpl.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/ClassReader.java
* tajo-common/src/main/java/org/apache/tajo/util/ProtoUtil.java
* tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java
* tajo-core/src/main/java/org/apache/tajo/webapp/HttpServer.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/JarOptimizer.java
* tajo-common/src/test/java/org/apache/tajo/datum/TestIntervalDatum.java
* 
tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Textifier.java
* tajo-core/src/main/java/org/apache/tajo/engine/codegen/CaseWhenEmitter.java


> 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
>             Fix For: 0.12.0
>
>
> 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)

Reply via email to