You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@solr.apache.org by no...@apache.org on 2022/11/13 23:07:10 UTC

[solr] branch main updated: Revert "SOLR-16512: Fix writing JSON objects in NoggitJSONWriter"

This is an automated email from the ASF dual-hosted git repository.

noble pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/solr.git


The following commit(s) were added to refs/heads/main by this push:
     new 322fa0d4df3 Revert "SOLR-16512: Fix writing JSON objects in NoggitJSONWriter"
322fa0d4df3 is described below

commit 322fa0d4df3519c75be1e73ac41eb01732c0e2dc
Author: Noble Paul <no...@gmail.com>
AuthorDate: Mon Nov 14 10:06:22 2022 +1100

    Revert "SOLR-16512: Fix writing JSON objects in NoggitJSONWriter"
    
    This reverts commit 364cdc87e3935ac5eb603d161fa82cdf968832bb.
    
    SOLR-16512: revert until perf issues are fixed
---
 solr/solrj/src/java/org/apache/solr/common/util/NoggitJSONWriter.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/solr/solrj/src/java/org/apache/solr/common/util/NoggitJSONWriter.java b/solr/solrj/src/java/org/apache/solr/common/util/NoggitJSONWriter.java
index 7dc474faa6b..d31e598604c 100644
--- a/solr/solrj/src/java/org/apache/solr/common/util/NoggitJSONWriter.java
+++ b/solr/solrj/src/java/org/apache/solr/common/util/NoggitJSONWriter.java
@@ -59,7 +59,7 @@ public class NoggitJSONWriter extends JSONWriter {
 
   private IteratorWriter.ItemWriter itemWriter() {
     return new IteratorWriter.ItemWriter() {
-      private boolean first = true;
+      private boolean first;
 
       @Override
       public IteratorWriter.ItemWriter add(Object o) {
@@ -77,7 +77,7 @@ public class NoggitJSONWriter extends JSONWriter {
 
   private MapWriter.EntryWriter entryWriter() {
     return new MapWriter.EntryWriter() {
-      private boolean first = true;
+      private boolean first;
 
       @Override
       public MapWriter.EntryWriter put(CharSequence k, Object v) {