alex-plekhanov commented on code in PR #10230:
URL: https://github.com/apache/ignite/pull/10230#discussion_r965761254
##########
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:
Javadoc for this parameter of the old method looks better to me, but it's up
to you.
Also, the first word of the javadoc should start with the upper case letter.
--
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]
