That’s not a plothole, and it doesn’t really have anything to do with the point I’m making regarding Finn’s suicide run.
Poe was reckless, and did endanger the fleet. Again, this is very clearly shown to us through visuals and dialog. He also reduced the number of threats that followed them through hyperspace, yes - but considering Snoke’s flagship showed up anyway, (a giant ship that literally dwarfed the dreadnought, and brought its own set of Star Destroyers with it) the idea that losing all those pilots and ships to destroy the one dreadnought was “worth it” in the way Poe thought it was is very clearly shown by the film to be not true.
But again: What happens to the story if that “plot hole” is patched up? What is that saying in the context of the larger story, and where is it pointing? If you’re “fixing” stuff without keeping any of that in mind, what are you actually “fixing?”
Poe’s arc is started by Paige’s sacrifice, and how that sacrifice wasn’t quite in vain, but was definitely unneccessary in the larger scheme. Having Poe be in charge of a similar situation on Crait, and recognizing his mistake this time, goes a long way towards resolving that arc later in the cave when Finn is, emotionally and character-wise, where Poe was.
And that’s how you wind up with Finn wanting to make Paige’s sacrifice, but this time even less effectively, with even more on the line, and being stopped by Paige’s sister from doing that so that both Finn and Poe can complete their character arcs in this story. That’s not a plot-hole.