noticket: [repository-inmemory] Deprecate DbMemory for removal

Author: nvamelichev

repository-inmemory/src/main/java/tech/ydb/yoj/repository/test/inmemory/InMemoryRepositoryTransaction.java

@@ -3,6 +3,7 @@
 import com.google.common.base.Stopwatch;
 import com.google.common.collect.Iterables;
 import lombok.Getter;
+import tech.ydb.yoj.DeprecationWarnings;
 import tech.ydb.yoj.repository.BaseDb;
 import tech.ydb.yoj.repository.db.Entity;
 import tech.ydb.yoj.repository.db.RepositoryTransaction;
@@ -57,16 +58,21 @@ private long getVersion() {
 
     @Override
     public <T extends Entity<T>> Table<T> table(Class<T> c) {
-        return new InMemoryTable<>(getMemory(c));
+        return new InMemoryTable<>(this, c);
     }
 
     @Override
     public <T extends Entity<T>> Table<T> table(TableDescriptor<T> tableDescriptor) {
         return new InMemoryTable<>(this, tableDescriptor);
     }
 
-    @Deprecated // use other constructor of InMemoryTable
+    /**
+     * @deprecated {@code DbMemory} and this method will be removed in YOJ 2.7.0.
+     */
+    @Deprecated(forRemoval = true)
     public final <T extends Entity<T>> InMemoryTable.DbMemory<T> getMemory(Class<T> c) {
+        DeprecationWarnings.warnOnce("InMemoryTable.getMemory(Class)",
+                "InMemoryTable.getMemory(Class<T>) will be removed in YOJ 2.7.0. Please stop using this method");
         return new InMemoryTable.DbMemory<>(c, this);
     }
 

repository-inmemory/src/main/java/tech/ydb/yoj/repository/test/inmemory/InMemoryTable.java

@@ -3,6 +3,7 @@
 import com.google.common.base.Preconditions;
 import com.google.common.collect.Iterables;
 import com.google.common.collect.Sets;
+import tech.ydb.yoj.DeprecationWarnings;
 import tech.ydb.yoj.databind.expression.FilterExpression;
 import tech.ydb.yoj.databind.expression.OrderExpression;
 import tech.ydb.yoj.databind.schema.ObjectSchema;
@@ -38,9 +39,15 @@ public class InMemoryTable<T extends Entity<T>> implements Table<T> {
     private final TableDescriptor<T> tableDescriptor;
     private final InMemoryRepositoryTransaction transaction;
 
-    @Deprecated // Don't use DbMemory, use other constructor instead
+    /**
+     * @deprecated {@code DbMemory} and this {@code InMemoryTable} constructor will be removed in YOJ 2.7.0.
+     * Please use other {@code InMemoryTable} constructors instead.
+     */
+    @Deprecated(forRemoval = true)
     public InMemoryTable(DbMemory<T> memory) {
         this(memory.transaction(), memory.type());
+        DeprecationWarnings.warnOnce("new InMemoryTable(DbMemory)",
+                "Please do not use the InMemoryTable(DbMemory<T>) constructor, it will be removed in YOJ 2.7.0");
     }
 
     public InMemoryTable(InMemoryRepositoryTransaction transaction, Class<T> type) {
@@ -586,8 +593,12 @@ private static <V extends Table.View, T extends Entity<T>> V toView(
         return Columns.fromEntity(schema, entity).toSchema(viewSchema);
     }
 
-
-    @Deprecated // Legacy. Using only for creating InMemoryTable. Use constructor of InMemoryTable instead
+    /**
+     * @deprecated Legacy class, used only for creating {@code InMemoryTable}.
+     * This class will be removed in YOJ 2.7.0.
+     * Please use the {@code InMemoryTable(InMemoryRepositoryTransaction, Class)} constructor instead of {@code InMemoryTable(DbMemory)} constructor.
+     */
+    @Deprecated(forRemoval = true)
     public record DbMemory<T extends Entity<T>>(
             Class<T> type,
             InMemoryRepositoryTransaction transaction

repository-inmemory/src/test/java/tech/ydb/yoj/repository/test/inmemory/TestInMemoryRepository.java

@@ -11,6 +11,7 @@
 import tech.ydb.yoj.repository.test.sample.TestEntityOperations.BubbleTable;
 import tech.ydb.yoj.repository.test.sample.TestEntityOperations.ComplexTable;
 import tech.ydb.yoj.repository.test.sample.TestEntityOperations.IndexedTable;
+import tech.ydb.yoj.repository.test.sample.TestEntityOperations.Supabubble2Table;
 import tech.ydb.yoj.repository.test.sample.model.Bubble;
 import tech.ydb.yoj.repository.test.sample.model.Complex;
 import tech.ydb.yoj.repository.test.sample.model.DetachedEntity;
@@ -105,7 +106,7 @@ public SupabubbleTable supabubbles() {
 
         @Override
         public Supabubble2Table supabubbles2() {
-            return new Supabubble2InMemoryTable(getMemory(Supabubble2.class));
+            return new Supabubble2InMemoryTable(table(Supabubble2.class));
         }
 
         @Override
@@ -144,9 +145,9 @@ public Table<MultiWrappedEntity2> multiWrappedEntities2() {
         }
     }
 
-    private static class Supabubble2InMemoryTable extends InMemoryTable<Supabubble2> implements TestEntityOperations.Supabubble2Table {
-        public Supabubble2InMemoryTable(DbMemory<Supabubble2> memory) {
-            super(memory);
+    private static class Supabubble2InMemoryTable extends AbstractDelegatingTable<Supabubble2> implements Supabubble2Table {
+        public Supabubble2InMemoryTable(Table<Supabubble2> target) {
+            super(target);
         }
     }