Fix issues that GroupBy and Sample doesn't call 'unsubscribe' and also NPE when the key is null in GroupBy

Fix #1958
> This will unsubscribe the downstream. I suggest instead having child.add(sampler); after L52.

Could you elaborate? I think `unsubscribe the downstream` would be OK for this operator.