[GitHub] [hudi] bvaradar commented on pull request #8385: [HUDI-6040]Stop writing and reading compaction plans from .aux folder

2023-04-28 Thread via GitHub
bvaradar commented on PR #8385: URL: https://github.com/apache/hudi/pull/8385#issuecomment-1527181265 @nsivabalan : Are you ok with this change ? If so, can you approve it ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [hudi] bvaradar commented on pull request #8385: [HUDI-6040]Stop writing and reading compaction plans from .aux folder

2023-04-24 Thread via GitHub
bvaradar commented on PR #8385: URL: https://github.com/apache/hudi/pull/8385#issuecomment-1521159303 > Should we completely remove all usages of "aux" folder with this patch? https://user-images.githubusercontent.com/513218/234157269-4e18e48c-2060-41af-8991-1ed9d15c5592.png";> > > I

[GitHub] [hudi] bvaradar commented on pull request #8385: [HUDI-6040]Stop writing and reading compaction plans from .aux folder

2023-04-19 Thread via GitHub
bvaradar commented on PR #8385: URL: https://github.com/apache/hudi/pull/8385#issuecomment-1515644714 @Mulavar : W.r.t running compaction after upgrade/downgrade, this will help prove that the upgrade will be safer if we can ensure compaction did not fail in any place after upgrade/downgrad

[GitHub] [hudi] bvaradar commented on pull request #8385: [HUDI-6040]Stop writing and reading compaction plans from .aux folder

2023-04-11 Thread via GitHub
bvaradar commented on PR #8385: URL: https://github.com/apache/hudi/pull/8385#issuecomment-1504389808 @Mulavar : We need to create a higher version 6 and write upgrade/downgrade to handle transition from current version (5). You can look at https://github.com/apache/hudi/pull/6248 as examp