palashc commented on code in PR #1866:
URL: https://github.com/apache/phoenix/pull/1866#discussion_r1621231370
##########
phoenix-core/src/it/java/org/apache/phoenix/end2end/index/BaseIndexIT.java:
##########
@@ -1514,7 +1514,6 @@ public void testLastDDLTimestampOnAsyncIndexes() throws
Exception {
// run the index MR job.
IndexToolIT.runIndexTool(false, TestUtil.DEFAULT_SCHEMA_NAME,
tableName, indexName);
- TestUtil.waitForIndexState(conn, fullIndexName,
PIndexState.ACTIVE);
Review Comment:
@haridsv Would you remember why this change was made in this PR? This breaks
a few tests on my PR https://github.com/apache/phoenix/pull/1883.
Rushabh made the
[change](https://github.com/apache/phoenix/pull/1595/files#diff-6d91af246300bfabfe5d43b94abe177fa549c3dc60d2e25e795e1425a0ff4bb4R1466)
originally and I think it is needed. Please let me know if there was a reason
to remove it in this PR.
--
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]
