Yes, you could do it that way but you would need to have a relationship field on each “Edition” post to connect to all the “Books”. If you don’t then you’d still be looking at doing a query of all the books and looping to find the right ones.
The main issue is that you have the “Edition” type in a repeater and it is difficult to search by a repeater sub field, there is some documentation about it on this page https://www.advancedcustomfields.com/resources/query-posts-custom-fields/. If you do this the performance will not be good.
