I don't understand this change. That's what `self.config.executor_identity()` returns. 

Or, is it a mistake to keep the `executor_identity` method?