HTHou commented on code in PR #18132:
URL: https://github.com/apache/iotdb/pull/18132#discussion_r3612979770
##########
iotdb-client/isession/src/main/java/org/apache/iotdb/isession/ITableSession.java:
##########
@@ -46,6 +48,16 @@ public interface ITableSession extends AutoCloseable {
*/
void insert(Tablet tablet) throws StatementExecutionException,
IoTDBConnectionException;
+ /**
+ * Inserts multiple {@link Tablet}s into the database.
+ *
+ * @param tablets the tablets containing time-series data to be inserted.
+ * @throws StatementExecutionException if an error occurs while executing
the statement.
+ * @throws IoTDBConnectionException if there is an issue with the IoTDB
connection.
+ */
+ void insertTablets(List<Tablet> tablets)
Review Comment:
**API suggestion:** `ITableSession` already exposes the single-tablet
operation as `insert(Tablet)`. Could this new batch method be named
`insert(List<Tablet>)` or `batchInsert(List<Tablet>)` as an overload, with the
corresponding implementations and call sites updated together? That would keep
the table-session API consistent while this method is still being introduced.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
