Github user Jiri-Kremser commented on the issue:
https://github.com/apache/spark/pull/19802
> I think it's probably OK if you also re-throw that exception for now.
You're just giving more info then.
Hmm, In the original code the exception wasn't re-thrown. Do you really
think it is a good idea? The `InvalidClassException` is a checked exception so
the signature of `processOneWayMessage()` method would need to be change as
well.
this is the original catch-them-all code :)
```java
} catch (Exception e) {
logger.error("Error while invoking RpcHandler#receive() for one-way
message.", e);
}
```
I think, I addressed all your inline other comments, thanks for the review.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
