You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by li...@apache.org on 2022/03/04 13:24:14 UTC

[calcite] 10/41: [CALCITE-4953] Deprecate TableAccessMap class

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

liyafan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git

commit 3579eec401b09fe64df9cccb6ab798c7b67ef2bd
Author: Stamatis Zampetakis <za...@gmail.com>
AuthorDate: Mon Dec 20 00:17:56 2021 +0200

    [CALCITE-4953] Deprecate TableAccessMap class
    
    The TableAccessMap class is not used anywhere in the project, it is
    untested, and hasn't received any real update since its addition in the
    project in 2012.
    
    Close apache/calcite#2675
---
 core/src/main/java/org/apache/calcite/plan/TableAccessMap.java | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/core/src/main/java/org/apache/calcite/plan/TableAccessMap.java b/core/src/main/java/org/apache/calcite/plan/TableAccessMap.java
index 98a4fa5..59dc10c 100644
--- a/core/src/main/java/org/apache/calcite/plan/TableAccessMap.java
+++ b/core/src/main/java/org/apache/calcite/plan/TableAccessMap.java
@@ -35,7 +35,12 @@ import java.util.Set;
 /**
  * <code>TableAccessMap</code> represents the tables accessed by a query plan,
  * with READ/WRITE information.
+ *
+ * @deprecated As of 1.30.0, if you need to know how tables in a plan are accessed you are
+ * encouraged to implement your own logic (using a RelNode visitor or other). The class is not used
+ * anywhere in the project and remains untested thus it is deprecated.
  */
+@Deprecated // to be removed before 2.0
 public class TableAccessMap {
   //~ Enums ------------------------------------------------------------------