[
https://issues.apache.org/jira/browse/PHOENIX-2733?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15176793#comment-15176793
]
Hudson commented on PHOENIX-2733:
---------------------------------
FAILURE: Integrated in Phoenix-master #1165 (See
[https://builds.apache.org/job/Phoenix-master/1165/])
PHOENIX-2733 Minor cleanup for improvements to CSV Bulk Loader (jtaylor: rev
952a01cce6d7d314ca26e89a398e75f38690cdad)
*
phoenix-core/src/test/java/org/apache/phoenix/mapreduce/FormatToKeyValueMapperTest.java
*
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/AbstractBulkLoadTool.java
*
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/bulkload/TargetTableRefFunctions.java
*
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/JsonToKeyValueMapper.java
*
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/FormatToBytesWritableMapper.java
*
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/FormatToKeyValueReducer.java
*
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/FormatToKeyValueMapper.java
*
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/util/PhoenixConfigurationUtil.java
*
phoenix-core/src/test/java/org/apache/phoenix/mapreduce/FormatToBytesWritableMapperTest.java
*
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/CsvToKeyValueMapper.java
> Minor cleanup for improvements to CSV Bulk Loader performance
> -------------------------------------------------------------
>
> Key: PHOENIX-2733
> URL: https://issues.apache.org/jira/browse/PHOENIX-2733
> Project: Phoenix
> Issue Type: Bug
> Reporter: Gabriel Reid
> Assignee: Sergey Soldatov
> Priority: Minor
> Fix For: 4.7.0
>
> Attachments: PHOENIX-2733-1.patch, PHOENIX-2733-2.patch,
> PHOENIX-2733-3.patch, PHOENIX-2733-4.patch
>
>
> Based on the improvements from PHOENIX-1973, there are a few small things
> that could use some touching up:
> * TargetTableRefFunctions#LOGICAN_NAMES_TO_JSON appears to be spelled
> incorrectly (should be LOGICAL_NAMES_TO_JSON), and I don’t understand why
> it’s a Function instead of just a static method (it matches other Functions
> in that class, but the general use of Functions in that way makes no sense)
> * FormatToKeyValueMapper should probably be renamed to accurately describe
> what it does, as well as definitely updating the class-level javadoc to
> explain what it does (i.e. it’s not creating KeyValues any more)
> * Pretty minor code format issues, such as lack of correct spacing in
> FormatToKeyValueMapper#findIndex and elsewhere, and the use of wildcard
> imports in FormatToKeyValueMapper
> * Minor nit, but why is TrustedByteArrayOutputStream being used in
> FormatToKeyValueMapper#writeAggregatedRow?
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
