Thought it was entertaining throughout, even Sam wasn't annoying. The giants looked brilliant. Ygritte's death was done very well, short enough, so that it wasn't cringeworthy and I liked that for a change, a death wasn't shocking. It just happened like it was always going to happen.
My main criticism (and it's more a general one for the whole season) would be that I'm not sure that I fully felt the importance of the battle at the beginning of the episode, which lead to the lack of tension. The fact that no one in Westeros really cared about the wall throughout this season, made it seem less significant than it should be. It's a huge army waiting on the other side and it's not even the most dangerous army north of the wall. Westeros is probably fecked if they get through, but somehow it didn't feel that way to me. Jon Snow standing up in a room with a few men of the nightwatch a few episodes back wasn't enough. They needed a bigger storyarc, maybe Jon sending someone down to convince some of the Stark bannerman to help them or convince Stannis to come to the wall or something like that. Instead of wasting time with that Bolton spy, they should have had Sam traveling South and give him something important to do. They also should have showed more of Mance and his plans. All we saw from the wildling side was Ygritte and everytime it was about her love and her revenge, not about the real threat. It all felt too isolated from the rest and therefore it lacked the emotional connection to all the other stories.
It was a brilliant battle though, amazing how good a job they did with the budget available. Some of the action scenes were mind-blowing for a tv series.