[
https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17730720#comment-17730720
]
ASF GitHub Bot commented on PHOENIX-6962:
-----------------------------------------
virajjasani commented on code in PR #1619:
URL: https://github.com/apache/phoenix/pull/1619#discussion_r1223535117
##########
phoenix-core/src/main/java/org/apache/phoenix/compile/DeleteCompiler.java:
##########
@@ -748,6 +757,14 @@ public Long getEstimateInfoTimestamp() throws SQLException
{
}
}
+ /**
+ * Implementation of MutationPlan that is selected if
+ * 1) there are mutable indexes presented for the table,
+ * 2) auto commit is on,
Review Comment:
`auto commit is on` => `auto commit is enabled as well as server side delete
mutations are enabled`
thanks @tkhurana
> Write javadoc for different mutation plans within DeleteCompiler.
> -----------------------------------------------------------------
>
> Key: PHOENIX-6962
> URL: https://issues.apache.org/jira/browse/PHOENIX-6962
> Project: Phoenix
> Issue Type: Bug
> Components: core
> Reporter: Rushabh Shah
> Assignee: Jing Yu
> Priority: Minor
>
> There are multiple mutation plan within DeleteCompiler:
> # MultiRowDeleteMutationPlan
> # SingleRowDeleteMutationPlan
> # ServerSelectDeleteMutationPlan
> # ClientSelectDeleteMutationPlan
> It doesn't have javadocs explaining which mutation plan is selected for a
> given delete statement.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
