I prefer to have  this renamed to findInvoicesByPartnerAndPeriod( ... ).  For starters if the period is one year, you would have to return multiple invoices.  Secondly the method could easily be used outside of a billing context.  Therefor Billing in the function name can be dropped