うみ、そら、という名前自体は双子の名付けとしてとても素敵です。
でも、「幸」は読みに関わってないので、せっかく意味を込めて付けたであろう「幸」が余計なもののように感じてしまいます。

あと、2さんの言うように、海幸は海の幸が思い浮かぶので、将来からかわれないかなと心配になります。

こだわりを持って付けるのであれば良いんじゃないかなと思いますが、海、空の一文字だけの方が周りからの印象は良いと思いますよ。