sanpwc commented on a change in pull request #103:
URL: https://github.com/apache/ignite-3/pull/103#discussion_r620220206
##########
File path:
modules/api/src/main/java/org/apache/ignite/table/manager/TableManager.java
##########
@@ -15,10 +15,42 @@
* limitations under the License.
*/
+/**
+ * Interface for manage tables.
+ */
package org.apache.ignite.table.manager;
+import java.util.List;
+import java.util.function.Consumer;
+import org.apache.ignite.configuration.schemas.table.TableChange;
+import org.apache.ignite.table.Table;
+
/**
* Interface that provides methods for managing tables.
*/
public interface TableManager {
Review comment:
Cause it's a part of ignite-api, should we use IgniteTables or Tables
instead of exposing therm Manager externally.
I mean that we still should have TableMangerImpl -> TableManager that will
implement IgniteTables.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
