jnturton commented on code in PR #2800: URL: https://github.com/apache/drill/pull/2800#discussion_r1189562746
########## exec/java-exec/src/main/java/org/apache/drill/exec/store/http/oauth/OAuthUtils.java: ########## @@ -36,6 +37,7 @@ public class OAuthUtils { private static final Logger logger = LoggerFactory.getLogger(OAuthUtils.class); + private static final ObjectMapper MAPPER = JacksonUtils.createObjectMapper(); Review Comment: Follow up: [this post](https://stackoverflow.com/a/3909846/1153953) suggests that global ObjectReader and ObjectWriter objects might be a better choice than a global ObjectMapper. -- 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: dev-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org