Thanks for reaching out to Q&A.
It is better to use CreateReplaySafeLogger method in both orchestrator and sub-orchestrator methods. We know that this method returns an instance of ILogger that is replay safe. There are chances that the sub-orchestrator will replay its activity functions so it is safe to use CreateReplaySafeLogger method to avoid duplicate logs.
I hope this helps!
Please 'Accept as answer' and ‘Upvote’ if it helped so that it can help others in the community looking for help on similar topics.