tkhurana commented on a change in pull request #961:
URL: https://github.com/apache/phoenix/pull/961#discussion_r522497471



##########
File path: 
phoenix-core/src/it/java/org/apache/phoenix/end2end/index/MutableIndexIT.java
##########
@@ -899,6 +899,39 @@ public void testUpdateNonIndexedColumn() throws Exception {
       }
   }
 
+    @Test
+    public void testDeleteCount_nonPK() throws Exception {
+        String schemaName = generateUniqueName();
+        String dataTableName = "TBL_" + generateUniqueName();

Review comment:
       I just wanted to make sure I didn't break anything for mutable indexes 
so added a test to verify. The bug only applies to immutable index but the code 
is shared between both immutable and mutable. I could carry all the tests. What 
do you suggest ?




----------------------------------------------------------------
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]


Reply via email to