Вот сейчас в целом норм, правда ты меня малеха не правильно понял с асинхронными задачами
Итерацию по plugin.getServer().getOnlinePlayers() (в теории) безопасно делать асинхронно, обернуть все в try-cath и плевать если оно "иногда" будет выбрасывать исключение "одновременной модификации" в очень редких случаях (я не помню/уверен касательно типа данных, но если это обычный список-массив - то можно получать по индексу, это всегда будет безопасно)
Тк обновления позиции даже в теории может происходить только в 1 месте и только в 1 потоке - никакие синхронизации не нужны (проверка - пнуть игрока если при клике в инвентарь он убежал слишком далеко не должна модифицировать, верно?)
Синхронизация нужна только на добавление / удаление новых позиций, чтобы твоя мапа, где хранятся данные не сдохла
Можно конечно поставить потоко-безопасную мапу, но у них скорость получения значительно ниже, а тк это будет самой частой операцией - лучше будет просто поставить блокировки на модификацию и итерацию (у тебя нет итераций)
Я бы поступил так - попробовал получить (потоко-безопасно), если результат - null, войти в блокировку, добавить (далее твой код). Когда игрок вышел - войти в блокировку и удалить
