So you’re creating a Subject ( share() ), then you’re creating one more subscription with concat.

So you’re creating a Subject ( share() ), then you’re creating one more subscription with concat. Redundant work, and memory for something so simple.

Moreover, it’s less readable for someone who isn’t super familiar with Rx.

Keep it simple.