tkhurana commented on code in PR #1531:
URL: https://github.com/apache/phoenix/pull/1531#discussion_r1063854857
##########
phoenix-core/src/main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java:
##########
@@ -815,6 +818,10 @@ private void
getCurrentRowStates(ObserverContext<RegionCoprocessorEnvironment> c
BatchMutateContext context) throws
IOException {
Set<KeyRange> keys = new HashSet<KeyRange>(context.rowsToLock.size());
context.dataRowStates = new HashMap<ImmutableBytesPtr, Pair<Put,
Put>>(context.rowsToLock.size());
+ if (context.hasUncoveredIndex && !context.hasGlobalIndex &&
!context.hasTransform
Review Comment:
@kadirozde If there are multiple columns part of an uncovered index row key
and we get a partial update which includes only a subset of those columns then
in this case we still need to read the data table row to correctly generate the
index row key.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
