J-Bakuli commented on code in PR #10230:
URL: https://github.com/apache/ignite/pull/10230#discussion_r965694603
##########
modules/core/src/main/java/org/apache/ignite/configuration/ThinClientConfiguration.java:
##########
@@ -37,9 +37,17 @@ public class ThinClientConfiguration {
/** Active compute tasks per connection limit. */
private int maxActiveComputeTasksPerConn =
DFLT_MAX_ACTIVE_COMPUTE_TASKS_PER_CONNECTION;
- /** If {@code true} sends a server exception stack trace to the client
side. */
+ /**
+ * @see new field serverExcStackTraceToClient.
+ * If {@code true} sends a server exception stack trace to the client side.
+ * @deprecated please now use new field serverExcStackTraceToClientbecause.
+ */
+ @Deprecated
private boolean sendServerExcStackTraceToClient;
+ /** If {@code true} a server exception stack trace is sent to the client
side. */
+ private boolean serverExcStackTraceToClient;
Review Comment:
Thank you, I have changed the code to be in line with your comments above.
That new field was added to be inserted into another constructor that is to
be used later when new pull requests remove deprecated fields / methods from
the code completely. That was my intention.
This PR and the ticket is to add set/get methods w/o overcomplication so
agree with you.
--
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]
