Method getRequiredAttributes from Impl return List<Attribute> so why you don't use only:
return attributesManagerImpl.getRequiredAttributes(sess, resource, serviceIds) ?
Is there some reason for using ArrayList<Attribute>( ... ) ?