add logger and comment for serviceimpl