add logging in service layer