dlmarion commented on a change in pull request #2169:
URL: https://github.com/apache/accumulo/pull/2169#discussion_r656161829
##########
File path:
server/manager/src/main/java/org/apache/accumulo/manager/tableOps/delete/DeleteTable.java
##########
@@ -41,6 +42,19 @@ public DeleteTable(NamespaceId namespaceId, TableId tableId)
{
@Override
public long isReady(long tid, Manager env) throws Exception {
+
+ // Before attempting to delete the table, cancel any running user
+ // compactions.
+ if (Utils.reserveNamespace(env, namespaceId, tid, false, true,
TableOperation.COMPACT_CANCEL)
Review comment:
I did not look into that behavior, I just implemented @ctubbsii
suggestion about moving the logic server-side. I don't know enough about FATE
to even know that issue would occur. Is there a way to test if this FATE
transaction has a read or write lock held already?
--
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]
