wzx140 commented on code in PR #5522: URL: https://github.com/apache/hudi/pull/5522#discussion_r870906031
########## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/HoodieAppendHandle.java: ########## @@ -502,12 +521,16 @@ private void writeToBuffer(HoodieRecord<T> record) { record.seal(); } // fetch the ordering val first in case the record was deflated. - final Comparable<?> orderingVal = record.getData().getOrderingValue(); - Option<IndexedRecord> indexedRecord = getIndexedRecord(record); + final Comparable<?> orderingVal = ((HoodieRecordPayload)record.getData()).getOrderingValue(); + Option<HoodieRecord> indexedRecord = prepareRecord(record); if (indexedRecord.isPresent()) { // Skip the ignored record. - if (!indexedRecord.get().equals(IGNORE_RECORD)) { - recordList.add(indexedRecord.get()); + try { + if (indexedRecord.isPresent() && !indexedRecord.get().isIgnoredRecord(tableSchema, config.getProps())) { + recordList.add(indexedRecord.get()); + } + } catch (IOException e) { + LOG.error("Error writing record " + indexedRecord.get(), e); Review Comment: isIgnoredRecord will deserializes(throw IOException) data to get indexRecord and check if it is EmptyRecord -- 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: commits-unsubscr...@hudi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org