nsivabalan commented on pull request #1704:
URL: https://github.com/apache/hudi/pull/1704#issuecomment-738561925
@vinothchandar : may I know whats the consensus here. Let me take a stab,
but do confirm the same.
Introduce a new class for payload class that supports this specific
ordering. This payload class will have the following api
```
Option<IndexedRecord> combineAndGetUpdateValue(IndexedRecord currentValue,
Schema schema, Map<String, String> props) throws IOException;
```
Will this extend from existing HoodieRecordPayload? bcoz, this the interface
used everywhere right?
Or did you intend a class as follows
```
class HoodieRecordOrderingPayload implements HoodieRecordPayload{
String orderingField; // value passed in via constructor.
Option<IndexedRecord> combineAndGetUpdateValue(IndexedRecord currentValue,
Schema schema) throws IOException{
// honor ordering value and return
}
.
.
}
----------------------------------------------------------------
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]
