[
https://issues.apache.org/jira/browse/PHOENIX-6437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17338701#comment-17338701
]
ASF GitHub Bot commented on PHOENIX-6437:
-----------------------------------------
xcangCRM commented on a change in pull request #1219:
URL: https://github.com/apache/phoenix/pull/1219#discussion_r625448015
##########
File path:
phoenix-core/src/it/java/org/apache/phoenix/replication/SystemCatalogWALEntryFilterIT.java
##########
@@ -281,4 +347,68 @@ private int getAndAssertTenantCountInEdit(WAL.Entry entry)
{
}
return entry;
}
+
+ public WAL.Entry getEntry(TableName tableName, Scan scan, boolean
addIndexedKeyValueCell)
Review comment:
Does this have to be public?
##########
File path:
phoenix-core/src/main/java/org/apache/phoenix/replication/SystemCatalogWALEntryFilter.java
##########
@@ -103,18 +104,20 @@ private boolean isTenantRowCellSystemChildLink(final Cell
cell) {
if (!isTenantRowCell) {
boolean isChildLink = CellUtil.matchingQualifier(
cell, PhoenixDatabaseMetaData.LINK_TYPE_BYTES);
- if (isChildLink) {
- if (CellUtil.matchingValue(cell, CHILD_TABLE_BYTES)) {
+ // Check if cell is of type LINK_TYPE with value 4 or DeleteFamily
+ if ((isChildLink && CellUtil.matchingValue(cell, CHILD_TABLE_BYTES)) ||
+ CellUtil.isDeleteFamily(cell) ) {
Review comment:
this logic means you will replicate whenever it's a DeleteFamily cell,
regardless of childlink or not. Is this the intention?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Delete marker for parent-child rows does not get replicated via
> SystemCatalogWALEntryFilter
> -------------------------------------------------------------------------------------------
>
> Key: PHOENIX-6437
> URL: https://issues.apache.org/jira/browse/PHOENIX-6437
> Project: Phoenix
> Issue Type: Bug
> Components: core
> Reporter: Ankit Jain
> Assignee: Ankit Jain
> Priority: Major
>
> As part of PHOENIX-3639 SystemCatalogWALEntryFilter was introduced to
> replicate tenant owned rows from system.catalog and ignore the non-tenant
> rows. During recent testing it was realized that delete markers for
> parent-child rows does not get replicated. As part of this Jira we want to
> fix that.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
