[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[
https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384800#comment-17384800
]
ASF GitHub Bot commented on HUDI-2203:
--
liujinhui1994 commented on a change in pull request #3317:
URL: https://github.com/apache/hudi/pull/3317#discussion_r673833060
##
File path:
hudi-common/src/main/java/org/apache/hudi/common/model/OverwriteWithLatestAvroPayload.java
##
@@ -49,7 +49,7 @@ public OverwriteWithLatestAvroPayload(Option
record) {
@Override
public OverwriteWithLatestAvroPayload
preCombine(OverwriteWithLatestAvroPayload another) {
// pick the payload with greatest ordering value
-if (another.orderingVal.compareTo(orderingVal) > 0) {
+if (another.orderingVal.compareTo(orderingVal) >= 0) {
return another;
Review comment:
> I actually have a similar PR #3267 but have no time to finish it yet, you
can take it.
OK, thanks I will close this PR
--
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
> When the orderingVal is the same, take the latest one
> -
>
> Key: HUDI-2203
> URL: https://issues.apache.org/jira/browse/HUDI-2203
> Project: Apache Hudi
> Issue Type: Improvement
>Reporter: liujinhui
>Priority: Major
> Labels: pull-request-available
> Fix For: 0.9.0
>
>
> OverwriteWithLatestAvroPayload.
> When the orderingVal is the same, select the later data
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[ https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384799#comment-17384799 ] ASF GitHub Bot commented on HUDI-2203: -- liujinhui1994 closed pull request #3317: URL: https://github.com/apache/hudi/pull/3317 -- 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 > When the orderingVal is the same, take the latest one > - > > Key: HUDI-2203 > URL: https://issues.apache.org/jira/browse/HUDI-2203 > Project: Apache Hudi > Issue Type: Improvement >Reporter: liujinhui >Priority: Major > Labels: pull-request-available > Fix For: 0.9.0 > > > OverwriteWithLatestAvroPayload. > When the orderingVal is the same, select the later data -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[ https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384788#comment-17384788 ] ASF GitHub Bot commented on HUDI-2203: -- danny0405 commented on pull request #3317: URL: https://github.com/apache/hudi/pull/3317#issuecomment-884051312 I actually have a similar PR https://github.com/apache/hudi/pull/3267 but have no time to finish it yet, you can take it. -- 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 > When the orderingVal is the same, take the latest one > - > > Key: HUDI-2203 > URL: https://issues.apache.org/jira/browse/HUDI-2203 > Project: Apache Hudi > Issue Type: Improvement >Reporter: liujinhui >Priority: Major > Labels: pull-request-available > Fix For: 0.9.0 > > > OverwriteWithLatestAvroPayload. > When the orderingVal is the same, select the later data -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[
https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384786#comment-17384786
]
ASF GitHub Bot commented on HUDI-2203:
--
danny0405 commented on a change in pull request #3317:
URL: https://github.com/apache/hudi/pull/3317#discussion_r673824944
##
File path:
hudi-common/src/main/java/org/apache/hudi/common/model/OverwriteWithLatestAvroPayload.java
##
@@ -49,7 +49,7 @@ public OverwriteWithLatestAvroPayload(Option
record) {
@Override
public OverwriteWithLatestAvroPayload
preCombine(OverwriteWithLatestAvroPayload another) {
// pick the payload with greatest ordering value
-if (another.orderingVal.compareTo(orderingVal) > 0) {
+if (another.orderingVal.compareTo(orderingVal) >= 0) {
return another;
Review comment:
The original logic is actually right, because the passed in `another` is
the old existing value.
--
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
> When the orderingVal is the same, take the latest one
> -
>
> Key: HUDI-2203
> URL: https://issues.apache.org/jira/browse/HUDI-2203
> Project: Apache Hudi
> Issue Type: Improvement
>Reporter: liujinhui
>Priority: Major
> Labels: pull-request-available
> Fix For: 0.9.0
>
>
> OverwriteWithLatestAvroPayload.
> When the orderingVal is the same, select the later data
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[ https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384787#comment-17384787 ] ASF GitHub Bot commented on HUDI-2203: -- hudi-bot edited a comment on pull request #3317: URL: https://github.com/apache/hudi/pull/3317#issuecomment-883950618 ## CI report: * 1279f1e8d8fb04b2f3de55fda8010fdf4097178d Azure: [FAILURE](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=1074) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run travis` re-run the last Travis build - `@hudi-bot run azure` re-run the last Azure build -- 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 > When the orderingVal is the same, take the latest one > - > > Key: HUDI-2203 > URL: https://issues.apache.org/jira/browse/HUDI-2203 > Project: Apache Hudi > Issue Type: Improvement >Reporter: liujinhui >Priority: Major > Labels: pull-request-available > Fix For: 0.9.0 > > > OverwriteWithLatestAvroPayload. > When the orderingVal is the same, select the later data -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[ https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384752#comment-17384752 ] ASF GitHub Bot commented on HUDI-2203: -- hudi-bot edited a comment on pull request #3317: URL: https://github.com/apache/hudi/pull/3317#issuecomment-883950618 ## CI report: * b4fd6b64bb1d56a35b06f0dcc7e8a7b18710bbf6 Azure: [FAILURE](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=1070) * 1279f1e8d8fb04b2f3de55fda8010fdf4097178d Azure: [PENDING](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=1074) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run travis` re-run the last Travis build - `@hudi-bot run azure` re-run the last Azure build -- 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 > When the orderingVal is the same, take the latest one > - > > Key: HUDI-2203 > URL: https://issues.apache.org/jira/browse/HUDI-2203 > Project: Apache Hudi > Issue Type: Improvement >Reporter: liujinhui >Priority: Major > Labels: pull-request-available > Fix For: 0.9.0 > > > OverwriteWithLatestAvroPayload. > When the orderingVal is the same, select the later data -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[ https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384751#comment-17384751 ] ASF GitHub Bot commented on HUDI-2203: -- hudi-bot edited a comment on pull request #3317: URL: https://github.com/apache/hudi/pull/3317#issuecomment-883950618 ## CI report: * b4fd6b64bb1d56a35b06f0dcc7e8a7b18710bbf6 Azure: [FAILURE](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=1070) * 1279f1e8d8fb04b2f3de55fda8010fdf4097178d UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run travis` re-run the last Travis build - `@hudi-bot run azure` re-run the last Azure build -- 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 > When the orderingVal is the same, take the latest one > - > > Key: HUDI-2203 > URL: https://issues.apache.org/jira/browse/HUDI-2203 > Project: Apache Hudi > Issue Type: Improvement >Reporter: liujinhui >Priority: Major > Labels: pull-request-available > Fix For: 0.9.0 > > > OverwriteWithLatestAvroPayload. > When the orderingVal is the same, select the later data -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[
https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384749#comment-17384749
]
ASF GitHub Bot commented on HUDI-2203:
--
liujinhui1994 commented on a change in pull request #3317:
URL: https://github.com/apache/hudi/pull/3317#discussion_r673784231
##
File path:
hudi-common/src/main/java/org/apache/hudi/common/model/OverwriteWithLatestAvroPayload.java
##
@@ -49,7 +49,7 @@ public OverwriteWithLatestAvroPayload(Option
record) {
@Override
public OverwriteWithLatestAvroPayload
preCombine(OverwriteWithLatestAvroPayload another) {
// pick the payload with greatest ordering value
-if (another.orderingVal.compareTo(orderingVal) > 0) {
+if (another.orderingVal.compareTo(orderingVal) > 0 ||
another.orderingVal.compareTo(orderingVal) == 0) {
Review comment:
hahah，ok
--
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
> When the orderingVal is the same, take the latest one
> -
>
> Key: HUDI-2203
> URL: https://issues.apache.org/jira/browse/HUDI-2203
> Project: Apache Hudi
> Issue Type: Improvement
>Reporter: liujinhui
>Priority: Major
> Labels: pull-request-available
> Fix For: 0.9.0
>
>
> OverwriteWithLatestAvroPayload.
> When the orderingVal is the same, select the later data
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[
https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384748#comment-17384748
]
ASF GitHub Bot commented on HUDI-2203:
--
wangxianghu commented on a change in pull request #3317:
URL: https://github.com/apache/hudi/pull/3317#discussion_r673783221
##
File path:
hudi-common/src/main/java/org/apache/hudi/common/model/OverwriteWithLatestAvroPayload.java
##
@@ -49,7 +49,7 @@ public OverwriteWithLatestAvroPayload(Option
record) {
@Override
public OverwriteWithLatestAvroPayload
preCombine(OverwriteWithLatestAvroPayload another) {
// pick the payload with greatest ordering value
-if (another.orderingVal.compareTo(orderingVal) > 0) {
+if (another.orderingVal.compareTo(orderingVal) > 0 ||
another.orderingVal.compareTo(orderingVal) == 0) {
Review comment:
use `>= 0`
--
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
> When the orderingVal is the same, take the latest one
> -
>
> Key: HUDI-2203
> URL: https://issues.apache.org/jira/browse/HUDI-2203
> Project: Apache Hudi
> Issue Type: Improvement
>Reporter: liujinhui
>Priority: Major
> Labels: pull-request-available
> Fix For: 0.9.0
>
>
> OverwriteWithLatestAvroPayload.
> When the orderingVal is the same, select the later data
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[ https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384732#comment-17384732 ] ASF GitHub Bot commented on HUDI-2203: -- hudi-bot edited a comment on pull request #3317: URL: https://github.com/apache/hudi/pull/3317#issuecomment-883950618 ## CI report: * b4fd6b64bb1d56a35b06f0dcc7e8a7b18710bbf6 Azure: [FAILURE](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=1070) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run travis` re-run the last Travis build - `@hudi-bot run azure` re-run the last Azure build -- 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 > When the orderingVal is the same, take the latest one > - > > Key: HUDI-2203 > URL: https://issues.apache.org/jira/browse/HUDI-2203 > Project: Apache Hudi > Issue Type: Improvement >Reporter: liujinhui >Priority: Major > Labels: pull-request-available > Fix For: 0.9.0 > > > OverwriteWithLatestAvroPayload. > When the orderingVal is the same, select the later data -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[ https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384712#comment-17384712 ] ASF GitHub Bot commented on HUDI-2203: -- liujinhui1994 commented on pull request #3317: URL: https://github.com/apache/hudi/pull/3317#issuecomment-883959527 @wangxianghu please help review -- 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 > When the orderingVal is the same, take the latest one > - > > Key: HUDI-2203 > URL: https://issues.apache.org/jira/browse/HUDI-2203 > Project: Apache Hudi > Issue Type: Improvement >Reporter: liujinhui >Priority: Major > Labels: pull-request-available > Fix For: 0.9.0 > > > OverwriteWithLatestAvroPayload. > When the orderingVal is the same, select the later data -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[ https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384709#comment-17384709 ] ASF GitHub Bot commented on HUDI-2203: -- hudi-bot edited a comment on pull request #3317: URL: https://github.com/apache/hudi/pull/3317#issuecomment-883950618 ## CI report: * b4fd6b64bb1d56a35b06f0dcc7e8a7b18710bbf6 Azure: [PENDING](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=1070) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run travis` re-run the last Travis build - `@hudi-bot run azure` re-run the last Azure build -- 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 > When the orderingVal is the same, take the latest one > - > > Key: HUDI-2203 > URL: https://issues.apache.org/jira/browse/HUDI-2203 > Project: Apache Hudi > Issue Type: Improvement >Reporter: liujinhui >Priority: Major > Labels: pull-request-available > Fix For: 0.9.0 > > > OverwriteWithLatestAvroPayload. > When the orderingVal is the same, select the later data -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[ https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384708#comment-17384708 ] ASF GitHub Bot commented on HUDI-2203: -- hudi-bot commented on pull request #3317: URL: https://github.com/apache/hudi/pull/3317#issuecomment-883950618 ## CI report: * b4fd6b64bb1d56a35b06f0dcc7e8a7b18710bbf6 UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run travis` re-run the last Travis build - `@hudi-bot run azure` re-run the last Azure build -- 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 > When the orderingVal is the same, take the latest one > - > > Key: HUDI-2203 > URL: https://issues.apache.org/jira/browse/HUDI-2203 > Project: Apache Hudi > Issue Type: Improvement >Reporter: liujinhui >Priority: Major > Labels: pull-request-available > Fix For: 0.9.0 > > > OverwriteWithLatestAvroPayload. > When the orderingVal is the same, select the later data -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HUDI-2203) When the orderingVal is the same, take the latest one
[ https://issues.apache.org/jira/browse/HUDI-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17384705#comment-17384705 ] ASF GitHub Bot commented on HUDI-2203: -- liujinhui1994 opened a new pull request #3317: URL: https://github.com/apache/hudi/pull/3317 ## *Tips* - *Thank you very much for contributing to Apache Hudi.* - *Please review https://hudi.apache.org/contributing.html before opening a pull request.* ## What is the purpose of the pull request OverwriteWithLatestAvroPayload. When the orderingVal is the same, select the later data ## Brief change log org.apache.hudi.common.model.OverwriteWithLatestAvroPayload.class org.apache.hudi.common.model.TestOverwriteWithLatestAvroPayload.class ## Committer checklist - [ ] Has a corresponding JIRA in PR title & commit - [ ] Commit message is descriptive of the change - [ ] CI is green - [ ] Necessary doc changes done or have another open PR - [ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA. -- 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 > When the orderingVal is the same, take the latest one > - > > Key: HUDI-2203 > URL: https://issues.apache.org/jira/browse/HUDI-2203 > Project: Apache Hudi > Issue Type: Improvement >Reporter: liujinhui >Priority: Major > Fix For: 0.9.0 > > > OverwriteWithLatestAvroPayload. > When the orderingVal is the same, select the later data -- This message was sent by Atlassian Jira (v8.3.4#803005)
