I mean, I can see where it adds something to Hagrid's background, but that could have been done in a much shorter scene, just summed up very quickly. It was a whole story arc that went nowhere; no one was changed for better or for worse because of it, and the only thing it really accomplished was getting rid of Umbridge (though that bit was pretty awesome). So I think if she'd spent less time on it it could have been better.