[GitHub] [hudi] danny0405 commented on a change in pull request #3267: [HUDI-2170] Always choose the latest record for HoodieRecordPayload

2021-07-15 Thread GitBox
danny0405 commented on a change in pull request #3267: URL: https://github.com/apache/hudi/pull/3267#discussion_r670935008 ## File path: hudi-common/src/main/java/org/apache/hudi/common/table/log/AbstractHoodieLogRecordScanner.java ## @@ -113,8 +120,11 @@ protected

[GitHub] [hudi] danny0405 commented on a change in pull request #3267: [HUDI-2170] Always choose the latest record for HoodieRecordPayload

2021-07-15 Thread GitBox
danny0405 commented on a change in pull request #3267: URL: https://github.com/apache/hudi/pull/3267#discussion_r670949047 ## File path: hudi-common/src/main/java/org/apache/hudi/common/model/OverwriteWithLatestAvroPayload.java ## @@ -90,6 +93,14 @@ protected boolean

[GitHub] [hudi] danny0405 commented on a change in pull request #3267: [HUDI-2170] Always choose the latest record for HoodieRecordPayload

2021-07-15 Thread GitBox
danny0405 commented on a change in pull request #3267: URL: https://github.com/apache/hudi/pull/3267#discussion_r670948572 ## File path: hudi-common/src/main/java/org/apache/hudi/common/model/OverwriteWithLatestAvroPayload.java ## @@ -49,7 +49,7 @@ public

[GitHub] [hudi] danny0405 commented on a change in pull request #3267: [HUDI-2170] Always choose the latest record for HoodieRecordPayload

2021-07-15 Thread GitBox
danny0405 commented on a change in pull request #3267: URL: https://github.com/apache/hudi/pull/3267#discussion_r670948572 ## File path: hudi-common/src/main/java/org/apache/hudi/common/model/OverwriteWithLatestAvroPayload.java ## @@ -49,7 +49,7 @@ public

[GitHub] [hudi] danny0405 commented on a change in pull request #3267: [HUDI-2170] Always choose the latest record for HoodieRecordPayload

2021-07-15 Thread GitBox
danny0405 commented on a change in pull request #3267: URL: https://github.com/apache/hudi/pull/3267#discussion_r670935008 ## File path: hudi-common/src/main/java/org/apache/hudi/common/table/log/AbstractHoodieLogRecordScanner.java ## @@ -113,8 +120,11 @@ protected

[GitHub] [hudi] danny0405 commented on a change in pull request #3267: [HUDI-2170] Always choose the latest record for HoodieRecordPayload

2021-07-15 Thread GitBox
danny0405 commented on a change in pull request #3267: URL: https://github.com/apache/hudi/pull/3267#discussion_r670926655 ## File path: hudi-common/src/main/java/org/apache/hudi/common/table/log/AbstractHoodieLogRecordScanner.java ## @@ -113,8 +120,11 @@ protected

[GitHub] [hudi] danny0405 commented on a change in pull request #3267: [HUDI-2170] Always choose the latest record for HoodieRecordPayload

2021-07-15 Thread GitBox
danny0405 commented on a change in pull request #3267: URL: https://github.com/apache/hudi/pull/3267#discussion_r670923302 ## File path: hudi-common/src/main/java/org/apache/hudi/common/model/OverwriteWithLatestAvroPayload.java ## @@ -90,6 +93,14 @@ protected boolean

[GitHub] [hudi] danny0405 commented on a change in pull request #3267: [HUDI-2170] Always choose the latest record for HoodieRecordPayload

2021-07-15 Thread GitBox
danny0405 commented on a change in pull request #3267: URL: https://github.com/apache/hudi/pull/3267#discussion_r670923142 ## File path: hudi-common/src/main/java/org/apache/hudi/common/model/OverwriteWithLatestAvroPayload.java ## @@ -90,6 +93,14 @@ protected boolean

[GitHub] [hudi] danny0405 commented on a change in pull request #3267: [HUDI-2170] Always choose the latest record for HoodieRecordPayload

2021-07-15 Thread GitBox
danny0405 commented on a change in pull request #3267: URL: https://github.com/apache/hudi/pull/3267#discussion_r670231077 ## File path: hudi-common/src/main/java/org/apache/hudi/common/model/OverwriteWithLatestAvroPayload.java ## @@ -49,7 +49,7 @@ public

[GitHub] [hudi] danny0405 commented on a change in pull request #3267: [HUDI-2170] Always choose the latest record for HoodieRecordPayload

2021-07-14 Thread GitBox
danny0405 commented on a change in pull request #3267: URL: https://github.com/apache/hudi/pull/3267#discussion_r670069995 ## File path: hudi-common/src/test/java/org/apache/hudi/common/model/TestOverwriteWithLatestAvroPayload.java ## @@ -72,6 +72,17 @@ public void