stoty commented on a change in pull request #933:
URL: https://github.com/apache/phoenix/pull/933#discussion_r509383290



##########
File path: phoenix-core/src/it/java/org/apache/phoenix/end2end/StoreNullsIT.java
##########
@@ -76,6 +81,15 @@
     private final String ddlFormat;
     
     private String dataTableName;
+
+    // In this class we depend on the major compaction to be done instantly
+    // so wo are overwriting the PHOENIX_MAX_LOOKBACK_AGE_CONF_KEY to 0
+    @BeforeClass

Review comment:
       I would expect the change in TestUtil to fix this case as well. 
   Why do we need to override PHOENIX_MAX_LOOKBACK_AGE_CONF_KEY here ?




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