I mostly agree. The alternative is to instantiate the client within the BreweryDBService object, which would then happen every time the search action is hit. Or in the BreweryDBService class. What do you think is best?