Виталий Качан

В плане удобства лучше вариант напрямую из приложении запросом - логика не разбивается на несколько мест и легче потом разбираться и править. По скорости здесь различий почти не будет.
Вот если бы у тебя была процедура, которая делает несколько запросов, что-то копирует, инсертит/апдейтит, в общем такая логика, которая бы потребовала несколько запросов вместо вызова функции/процедуры, то в таком случае конечно процедура/функция значительно быстрее отработает и тут уже баланс между удобством для сопровождения проекта и скоростью выполнения запросов.

А что Debug метода DeleteAsync говорит? Entity находит или null? А смотришь в той же БД, или может ConnectionString разные?

Это аналогично защите от DDOS атак, погугли, есть существующие (платные) решения, но я бы не заморачивался, т.к. самописно эта проблема решается очень уж сложно и стоит посчитать, сколько денег потеряет бизнес за предполагаемое время DDOS-а и сколько стоят готовые решения/свой костыль.

Работать с БД не как localbd, а вынесенной на отдельный сервер и подключаться по IP

Есть опция для ключей в том же EntityFramework SetNull, ну или в схеме БД выставить для ключа SetNull
https://docs.microsoft.com/en-us/ef/core/saving/ca...

CreateMap<OrderCreateModel, OrderDto>()
                .ForMember(property1 => property1.UserId, 
                    property1 => property1
                        .MapFrom(property2 => property2.UserId));

В общем у тебя property дублируется название и не оттуда UserId берется.