[
https://issues.apache.org/jira/browse/PHOENIX-2630?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15122476#comment-15122476
]
Hudson commented on PHOENIX-2630:
---------------------------------
SUCCESS: Integrated in Phoenix-master #1112 (See
[https://builds.apache.org/job/Phoenix-master/1112/])
PHOENIX-2630 Ensure commit fails if mutable secondary index write fails
(jtaylor: rev c9594e83e144ebc3830922401de2addb3df0c81d)
*
phoenix-core/src/main/java/org/apache/phoenix/hbase/index/write/KillServerOnFailurePolicy.java
*
phoenix-core/src/main/java/org/apache/phoenix/index/PhoenixIndexFailurePolicy.java
*
phoenix-core/src/it/java/org/apache/phoenix/end2end/index/MutableIndexFailureIT.java
> Ensure commit fails if mutable secondary index write fails
> ----------------------------------------------------------
>
> Key: PHOENIX-2630
> URL: https://issues.apache.org/jira/browse/PHOENIX-2630
> Project: Phoenix
> Issue Type: Bug
> Reporter: James Taylor
> Assignee: James Taylor
> Priority: Blocker
> Fix For: 4.7.0
>
> Attachments: PHOENIX-2630.patch
>
>
> See MutableIndexFailureIT. This commit removed the fail() after the commit()
> which is wrong:
> https://github.com/apache/phoenix/commit/3d69fa21123d182577a58bbc517d40ea9dc5a2cd#diff-de5444255a9f79a58ad86932fd6a2afd
> We need to add it back and confirm that the test still passes. If the index
> write fails, the commit needs to throw - otherwise the client won't know
> there was a failure.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
