timoninmaxim commented on code in PR #12464:
URL: https://github.com/apache/ignite/pull/12464#discussion_r2499552927


##########
modules/rest-http/src/main/java/org/apache/ignite/internal/processors/rest/protocols/http/jetty/GridJettyRestProtocol.java:
##########
@@ -71,6 +72,9 @@ public class GridJettyRestProtocol extends 
GridRestProtocolAdapter {
         }
     }
 
+    /** Object mapper class name. */
+    private static final String JACKSON_DATABIND_OBJECT_MAPPER = 
"com.fasterxml.jackson.databind.ObjectMapper";

Review Comment:
   You should check "org.apache.ignite.internal.jackson.IgniteObjectMapper". We 
should avoid scenario when jackson will somehow part of another module. In this 
case we will risk get ClassNotFoundException in constructor 
GridJettyRestHandler.



-- 
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]

Reply via email to