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

maghamravikiran commented on PHOENIX-2538:
------------------------------------------

[~gabriel.reid] 
    Thanks for the review.  I have reverted my change on removing the status of 
delete in the new patch attached. 
    I did a cross check with an old version of  CsvBulkLoadTool.java and the 
import order matches with the one i have .  :)

[1] 
https://github.com/apache/phoenix/blob/4.3/phoenix-core/src/main/java/org/apache/phoenix/mapreduce/CsvBulkLoadTool.java

> CsvBulkLoadTool should return non-zero exit status if import fails
> ------------------------------------------------------------------
>
>                 Key: PHOENIX-2538
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-2538
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Gabriel Reid
>            Assignee: maghamravikiran
>             Fix For: 4.7.0
>
>         Attachments: PHOENIX-2538-1.patch, PHOENIX-2538.patch
>
>
> The changes in PHOENIX-2216 accidentally changed CsvBulkLoadTool so that it 
> does not correctly return a non-zero error code if the import job fails. This 
> makes it impossible for users of the tool to automatically determine if the 
> tool failed (e.g. when running it from shell scripts).



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

Reply via email to