[GitHub] [pulsar] lhotari commented on a diff in pull request #20428: [improve][broker] Remove uneffective solution for reducing GC pressure

["lhotari (via GitHub)" <gi...@apache.org>]

lhotari commented on code in PR #20428:
URL: https://github.com/apache/pulsar/pull/20428#discussion_r1211706779


##########
pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/stats/PublisherStatsImpl.java:
##########
@@ -53,35 +53,13 @@ public class PublisherStatsImpl implements PublisherStats {
     public boolean supportsPartialProducer;
 
     /** Producer name. */
-    @JsonIgnore
-    private int producerNameOffset = -1;
-    @JsonIgnore
-    private int producerNameLength;
-
+    private String producerName;

Review Comment:
   The former `@JsonIgnore` annotations were needed because of the extra ordinary solution to store String values. They aren't needed anymore.



-- 
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: commits-unsubscribe@pulsar.apache.org

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