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

janhoy 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 b42a0a73a84 Remove deprecated methods and fields in Tuple class (#1566)
b42a0a73a84 is described below

commit b42a0a73a84a58ec75a489973b24d7fba1208a29
Author: Jan Høydahl <[email protected]>
AuthorDate: Tue Apr 25 20:49:02 2023 +0200

    Remove deprecated methods and fields in Tuple class (#1566)
---
 solr/CHANGES.txt                                   |  2 ++
 .../org/apache/solr/client/solrj/io/Tuple.java     | 34 +++-------------------
 2 files changed, 6 insertions(+), 30 deletions(-)

diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index 55799396e35..79456936332 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -178,6 +178,8 @@ Other Changes
 
 * SOLR-16745: Consolidate DelegatedSolrQueryRequest and 
DelegatingSolrQueryRequest (Kevin Risden)
 
+* PR#1566: Removed deprecated members from Tuple class (janhoy)
+
 ==================  9.2.0 ==================
 
 New Features
diff --git 
a/solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/Tuple.java 
b/solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/Tuple.java
index 28d680e16b9..a13a90831c5 100644
--- a/solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/Tuple.java
+++ b/solr/solrj-streaming/src/java/org/apache/solr/client/solrj/io/Tuple.java
@@ -48,24 +48,9 @@ public class Tuple implements Cloneable, MapWriter {
    */
   public boolean EXCEPTION;
 
-  /**
-   * Tuple fields.
-   *
-   * @deprecated use {@link #getFields()} instead of this public field.
-   */
-  @Deprecated public Map<String, Object> fields = CollectionUtil.newHashMap(2);
-  /**
-   * External serializable field names.
-   *
-   * @deprecated use {@link #getFieldNames()} instead of this public field.
-   */
-  @Deprecated public List<String> fieldNames;
-  /**
-   * Mapping of external field names to internal tuple field names.
-   *
-   * @deprecated use {@link #getFieldLabels()} instead of this public field.
-   */
-  @Deprecated public Map<String, String> fieldLabels;
+  private final Map<String, Object> fields = CollectionUtil.newHashMap(2);
+  private List<String> fieldNames;
+  private Map<String, String> fieldLabels;
 
   public Tuple() {
     // just an empty tuple
@@ -240,16 +225,6 @@ public class Tuple implements Cloneable, MapWriter {
     return this.fields;
   }
 
-  /**
-   * Return all tuple fields.
-   *
-   * @deprecated use {@link #getFields()} instead.
-   */
-  @Deprecated(since = "8.6.0")
-  public Map<String, Object> getMap() {
-    return this.fields;
-  }
-
   /**
    * This represents the mapping of external field labels to the tuple's 
internal field names if
    * they are different from field names.
@@ -299,8 +274,7 @@ public class Tuple implements Cloneable, MapWriter {
 
   @Override
   public Tuple clone() {
-    Tuple clone = new Tuple(this);
-    return clone;
+    return new Tuple(this);
   }
 
   /**

Reply via email to