Remove nested transactions from domain (un)locking

[gbrodman]

This change is 

[jianglai]

Reviewed 1 of 2 files at r1, all commit messages.
Reviewable status: 1 of 2 files reviewed, 1 unresolved discussion (waiting on @gbrodman)

---

core/src/main/java/google/registry/tools/LockOrUnlockDomainCommand.java line 82 at r1 (raw file):

    ImmutableSet.Builder<String> successfulDomainsBuilder = new ImmutableSet.Builder<>();
    ImmutableMap.Builder<String, String> failedDomainsToReasons = new ImmutableMap.Builder<>();
    partition(getDomains(), BATCH_SIZE)

Is there a concern about removing batching here?

[gbrodman]

Reviewable status: 1 of 2 files reviewed, 1 unresolved discussion (waiting on @jianglai)

---

core/src/main/java/google/registry/tools/LockOrUnlockDomainCommand.java line 82 at r1 (raw file):

Previously, jianglai (Lai Jiang) wrote…

Is there a concern about removing batching here?

nah, we never do these in bulk anyway and this is just making it have a batch size of 1

[jianglai]

Reviewed 1 of 2 files at r1.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @gbrodman)