tkalkirill commented on code in PR #4887:
URL: https://github.com/apache/ignite-3/pull/4887#discussion_r1883497689


##########
modules/network-annotation-processor/src/main/java/org/apache/ignite/internal/network/processor/messages/MessageImplGenerator.java:
##########
@@ -137,12 +139,20 @@ public TypeSpec generateMessageImpl(MessageClass message, 
TypeSpec builderInterf
 
             if (getter.getAnnotation(IgniteToStringExclude.class) == null) {
                 IgniteToStringInclude includeAnnotation = 
getter.getAnnotation(IgniteToStringInclude.class);
-
-                AnnotationSpec includeAnnotationSpec = includeAnnotation == 
null
-                        ? 
AnnotationSpec.builder(IgniteToStringInclude.class).build()
-                        : AnnotationSpec.get(includeAnnotation);
-
-                fieldBuilder.addAnnotation(includeAnnotationSpec);
+                IgniteStringifier stringifierAnnotation = 
getter.getAnnotation(IgniteStringifier.class);
+
+                if (stringifierAnnotation != null) {

Review Comment:
   This is how it is intended, the user himself will create the string for 
output, without automatic hiding of sensitive data.



-- 
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: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to