J-Bakuli commented on code in PR #10230:
URL: https://github.com/apache/ignite/pull/10230#discussion_r965680946
##########
modules/core/src/main/java/org/apache/ignite/configuration/ThinClientConfiguration.java:
##########
@@ -103,22 +112,45 @@ public ThinClientConfiguration
setMaxActiveComputeTasksPerConnection(int maxActi
/**
* @return If {@code true} sends a server exception stack to the client
side.
+ * @deprecated please now use new method
getServerExceptionStackTraceToClient().
*/
+ @Deprecated
public boolean sendServerExceptionStackTraceToClient() {
return sendServerExcStackTraceToClient;
}
/**
* @param sendServerExcStackTraceToClient If {@code true} sends a server
exception stack to the client side.
* @return {@code this} for chaining.
+ * @deprecated please now use new method
setServerExceptionStackTraceToClient().
*/
+ @Deprecated
public ThinClientConfiguration
sendServerExceptionStackTraceToClient(boolean sendServerExcStackTraceToClient) {
this.sendServerExcStackTraceToClient = sendServerExcStackTraceToClient;
return this;
}
- /** {@inheritDoc} */
+ /**
+ * @return If {@code true} gets a server exception stack that is to be
sent to the client side.
+ */
+ public boolean getServerExceptionStackTraceToClient() {
+ return serverExcStackTraceToClient;
+ }
+
+ /**
+ * @param serverExcStackTraceToClient if {@code true} sets a server
exception stack that is to be sent to the client side.
+ * @return {@code this} for chaining.
+ */
+ public ThinClientConfiguration
setServerExceptionStackTraceToClient(boolean serverExcStackTraceToClient) {
Review Comment:
Agree, I have changed the method name as per your recommendation above.
Java doc then is "@param sendStackTrace if {@code true} sets a server to
client exception stack trace sending."
Do you agree?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
