It ruins that whole scene.

I wouldn't go quite that far, but it didn't help. The sequence highlighted the importance of their meeting, while suggesting a mystery surrounding his reaction. They seemed to really wanted to emphasize how important and uncertain what happens next will be.

You're right thought, the helicopter shot was a bit jarring. The audience had been content slowly following Rey up, our anticipation building with her climb, the momentous nature of it perhaps becoming clearer perhaps as the shots widened. The sudden close ups were perhaps also confusing and jarring after the wides, perhaps all mixed haphazardly in an inconsistent shot scheme. It climaxes rather abruptly at the top, except it didn't stop right there, where they might have more profitably ended it. The audiences interest had been focusing the keener the closer we approached, and then we're suddenly whirling around and away, as if it's unimportant, like we're some gull.

In retrospect, it might have worked in reverse. Start with the wide shots for the epic scenic cenematography. Show the Falcon coming in wide and landing, then slowly pan in. Maybe follow along behind Reys path, showing her ascent. As she climbs, you could gradually get closer and closer, it would build the tension and importance of the moment as well as make it more intimate. Finally at the top, end with the closest shots.

As it is, it seemed more than a little stilted and unnatural how long she must have stood there holding the lightsaber/prop standing stationary, which took me out of it a bit. It just seemed a really odd place to cut, for mostly apparently cinematic purposes, maybe even a little to the detriment of the characters and story. It's obviously meant to be a cliffhanger, but that shot seems to detract from that effect, in my present opinion. Maybe they were worried about pulling a Soprano Finale, and being criticized for cutting to black too abruptly.