After seeing the change to the interface, I know why you do so, because you want to set the default parameter value of 'filterToPackageSet'. This makes the method signature looks odd. Is it really useful to provide default parameter value here? If it's not, I suggest keeping the old method signature.