[GitHub] [hudi] xushiyan commented on pull request #2845: [HUDI-1723] Fix path selector listing files with the same mod date

2021-05-13 Thread GitBox
xushiyan commented on pull request #2845: URL: https://github.com/apache/hudi/pull/2845#issuecomment-841005896 @nsivabalan @vinothchandar added the UT in 98dfb4fd6dc50f0e0b6b3818686bdcab8aa4f0e8. Please review. Thanks. -- This is an automated message from the Apache Git Service. To respo

[GitHub] [hudi] xushiyan commented on pull request #2845: [HUDI-1723] Fix path selector listing files with the same mod date

2021-05-10 Thread GitBox
xushiyan commented on pull request #2845: URL: https://github.com/apache/hudi/pull/2845#issuecomment-837449159 @vinothchandar sounds good. i'll add some tests in these 2 days -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [hudi] xushiyan commented on pull request #2845: [HUDI-1723] Fix path selector listing files with the same mod date

2021-04-21 Thread GitBox
xushiyan commented on pull request #2845: URL: https://github.com/apache/hudi/pull/2845#issuecomment-824535848 @nsivabalan thanks for reviewing this...I've listed my points in #2850. will start work on tests once we have an agreed plan. -- This is an automated message from the Apache Git

[GitHub] [hudi] xushiyan commented on pull request #2845: [HUDI-1723] Fix path selector listing files with the same mod date

2021-04-18 Thread GitBox
xushiyan commented on pull request #2845: URL: https://github.com/apache/hudi/pull/2845#issuecomment-822141919 @vinothchandar This is the short-term fix for the bug. Logic were duplicated due to the existing logic copied over from one to another. -- This is an automated message from the