keith-turner opened a new issue, #5724: URL: https://github.com/apache/accumulo/issues/5724
**Is your feature request related to a problem? Please describe.** After the changes in #5451 the mapping of table id to table name went from being stored in many nodes to being stored in a single node in zookeeper. This change allows detection of concurrent operations trying to create a table with the same name to rely on zookeeper. Therefore it seems like this [lock](https://github.com/apache/accumulo/blob/9749dc28be13838a4c4846db8e71af602caae1ea/server/manager/src/main/java/org/apache/accumulo/manager/tableOps/create/PopulateZookeeper.java#L54) is no longer needed. Prior to #5451 that lock was needed because multiple nodes had to be read in zookeeper to see if the name existed. **Describe the solution you'd like** Remove the lock if it seems zookeeper can now defend against concurrent create table operation that are trying to create a table with the same name. If it does not exists, it may be worthwhile to create a new test that spins up N threads all trying to create a table with the same name. Only one thread should succeed and all others should fail. -- 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]
