beryllw commented on code in PR #2161:
URL: https://github.com/apache/fluss/pull/2161#discussion_r2654572783
##########
fluss-server/src/main/java/org/apache/fluss/server/kv/KvTablet.java:
##########
@@ -468,8 +474,9 @@ private long applyInsert(
PaddingRow latestSchemaRow,
long logOffset)
throws Exception {
- walBuilder.append(ChangeType.INSERT,
latestSchemaRow.replaceRow(currentValue.row));
- kvPreWriteBuffer.put(key, currentValue.encodeValue(), logOffset);
+ BinaryValue newValue = autoIncProcessor.processAutoInc(currentValue);
Review Comment:
Currently, auto-increment is only utilized for INSERT operations, which does
not overlap with the functionalities of RowMerger and PartialUpdateRowMerger. I
agree integrating auto-increment into RowMerger would result in a more
streamlined design, and I will try to optimize this part.
--
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]
