[
https://issues.apache.org/jira/browse/PHOENIX-2374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15033112#comment-15033112
]
Hudson commented on PHOENIX-2374:
---------------------------------
FAILURE: Integrated in Phoenix-master #993 (See
[https://builds.apache.org/job/Phoenix-master/993/])
PHOENIX-2374 Honor PTable.isWALDisabled() for row deletes (jtaylor: rev
15ab697fa0958424a165862de411a8253e184270)
* phoenix-core/src/main/java/org/apache/phoenix/schema/PTableImpl.java
* phoenix-core/src/test/java/org/apache/phoenix/schema/MutationTest.java
> Honor PTable.isWALDisabled() for row deletes
> --------------------------------------------
>
> Key: PHOENIX-2374
> URL: https://issues.apache.org/jira/browse/PHOENIX-2374
> Project: Phoenix
> Issue Type: Bug
> Reporter: James Taylor
> Assignee: James Taylor
> Fix For: 4.7.0
>
> Attachments: PHOENIX-2374.patch
>
>
> Just noticed this incorrect code in PTableImpl.PRowImpl.delete():
> {code}
> @Override
> public void delete() {
> newMutations();
> Delete delete = new Delete(key);
> for (PColumnFamily colFamily : families) {
> delete.deleteFamily(colFamily.getName().getBytes(), ts);
> }
> deleteRow = delete;
> // No need to write to the WAL for indexes
> if (PTableImpl.this.getType() == PTableType.INDEX) {
> deleteRow.setDurability(Durability.SKIP_WAL);
> }
> }
> {code}
> We shouldn't special case indexes, but just use the standard isWALDisabled()
> check here like this:
> {code}
> @Override
> public void delete() {
> newMutations();
> Delete delete = new Delete(key);
> for (PColumnFamily colFamily : families) {
> delete.deleteFamily(colFamily.getName().getBytes(), ts);
> }
> deleteRow = delete;
> if (PTableImpl.this.isWALDisabled()) {
> deleteRow.setDurability(Durability.SKIP_WAL);
> }
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
