stub: add ServerCallStreamObserver.setOnCloseHandler(...) (#5895) - #8452

Finish does look better than Success. Thanks for the name brainstorming. We'll choose a name in the API meeting.

@ejona86
I've cleaned and rebuilt everything again, but cannot reproduce the error that occurred in the checks. Furthermore, looking at the output it does not seem to be related to my changes:

Task :grpc-netty:test

io.grpc.netty.AdvancedTlsTest > onFileReloadingKeyManagerTrustManagerTest FAILED
java.lang.AssertionError: Find error: UNAVAILABLE: io exception
Channel Pipeline: [SslHandler#0, ProtocolNegotiators$ClientTlsHandler#0, WriteBufferingAndExceptionHandler#0, DefaultChannelPipeline$TailContext#0]
at org.junit.Assert.fail(Assert.java:88)
at io.grpc.netty.AdvancedTlsTest.onFileReloadingKeyManagerTrustManagerTest(AdvancedTlsTest.java:385)

any hints how to deal with it? ;-)

@morgwai, don't worry about it. #8454

if #8476 is indeed a bug (as opposed to documentation omission, that I suspect), then onCloseHandler may be useful also for streaming clients, so that they do not exit before trailing metadata is transffered. In such case setOnCloseHandler should probably be moved to CallStreamObserver.

then onCloseHandler may be useful also for streaming clients

The onCloseHandler is powered by ServerCall.Listener.onComplete(). On client-side, the equivalent would be ClientCall.Listener.onClose(). However, ClientCall.Listener.onClose() already calls StreamObserver.onCompleted() or StreamObserver.onError(), depending on the status code. So on client-side the equivalent callback(s) already exist.

@ejona86 I've applied most of your comments. Please have a look at the one I was not sure about if you like the way it is now.

@ejona86 friendly ping :)
Please let me know if the current doc is ok or if you want me to change it somehow. Thanks!

Oh, thanks for the ping. Yes, this had been lost. I'll try and take a look tomorrow.

Thank you, @morgwai! Thanks for putting up with the process.

@ejona86 my pleasure :) thanks for the review!