You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@iotdb.apache.org by GitBox <gi...@apache.org> on 2021/09/26 07:49:55 UTC
[GitHub] [iotdb] HTHou commented on a change in pull request #3917: [IOTDB-1634][IOTDB-1571] Extend more aligned timeseries insert APIs && Fix aligned timeseries wal cannot recover
HTHou commented on a change in pull request #3917:
URL: https://github.com/apache/iotdb/pull/3917#discussion_r716160471
##########
File path: session/src/main/java/org/apache/iotdb/session/Session.java
##########
@@ -861,25 +861,31 @@ public void insertRecord(
long time,
List<String> measurements,
List<TSDataType> types,
- List<Object> values,
- boolean isAligned)
+ List<Object> values)
throws IoTDBConnectionException, StatementExecutionException {
+ // not vector by default
TSInsertRecordReq request =
- genTSInsertRecordReq(deviceId, time, measurements, types, values, isAligned);
+ genTSInsertRecordReq(deviceId, time, measurements, types, values, false);
insertRecord(deviceId, request);
}
- public void insertRecord(
- String deviceId,
+ /**
+ * insert aligned data in one row, if you want improve your performance, please use
+ * insertAlignedRecords method or insertTablet method.
+ *
+ * @see Session#insertRecords(List, List, List, List, List)
Review comment:
Fixed
--
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: reviews-unsubscribe@iotdb.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org