I loved everything with Luke/Rey/Kylo. Killing Snoke was the best thing ever (Snoke was so, sooo lame), the final lightsaber duel was AWESOME, and Luke's death was perfect.
The rest of the movie had a bunch of cool moments (the Holdo Maneuver was an awesome moment, no matter how little sense it makes), but was structurally awkward.
This movie, ultimately, needed another edit. Of note:
1) Space Battle #1 (w/Bombers) and Space Battle #2 (w/Leia getting blown up) could have been combined.
2) I don't know why we blew up Leia except just to get Liea on the Salt Planet to say bye to Luke. There was no other reason. She and Holdo are basically the same character in most ways, and Leia blowing the crap out of the First Order in a suicide charge would have been BAD ASS. Of course, they wanted Fisher in Ep9, so they kept her around, but still Holdo was this weird Leia doppleganger that I thought a little awkward.
3) There is zero (zero) reason they needed that whole rigamarole w/ the special "we track you through hyperspace" tech. All of it could have been managed with a mole on the ship (which they kinda hinted at anyway) or any number of other solutions that wouldn't have required us to take a weird side-quest to a casino planet.
4) Regarding Canto Bight, I thought it was a perfectly reasonable sequence (though the space pony chase went too long), but super, suuuper awkwardly inserted into the movie. Why couldn't the whole fleet have jumped *to* Canto Bight (in hopes of stealing/buying supplies while hiding in the SUPER CONVENIENT ASTEROID BELT I'M JUST NOW INVENTING)? That would have been more streamlined than them stealing a ship (somehow) to run off to another world and then jumping back again.
Overall, I enjoyed the hell out of the movie (it's ranking somewhere around #2 or #3 of all time Star Wars favorites), but it was really weirdly put together in a way that made things seem arbitrary.