[GitHub] [hudi] weimingdiit commented on pull request #7362: [HUDI-5315] The record size is dynamically estimated when the table i…

2023-04-11 Thread via GitHub
weimingdiit commented on PR #7362: URL: https://github.com/apache/hudi/pull/7362#issuecomment-1502785581 > > > @weimingdiit thanks for making the patch. I see the main problem here is that it's using in-memory size for the estimation which is actually intended for storage size, which may no

[GitHub] [hudi] weimingdiit commented on pull request #7362: [HUDI-5315] The record size is dynamically estimated when the table i…

2023-04-06 Thread via GitHub
weimingdiit commented on PR #7362: URL: https://github.com/apache/hudi/pull/7362#issuecomment-1498766781 > @weimingdiit thanks for making the patch. I see the main problem here is that it's using in-memory size for the estimation which is actually intended for storage size, which may not be

[GitHub] [hudi] weimingdiit commented on pull request #7362: [HUDI-5315] The record size is dynamically estimated when the table i…

2023-04-04 Thread via GitHub
weimingdiit commented on PR #7362: URL: https://github.com/apache/hudi/pull/7362#issuecomment-1495515476 @hudi-bot run azure -- 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.

[GitHub] [hudi] weimingdiit commented on pull request #7362: [HUDI-5315] The record size is dynamically estimated when the table i…

2023-03-15 Thread via GitHub
weimingdiit commented on PR #7362: URL: https://github.com/apache/hudi/pull/7362#issuecomment-1471243966 @hudi-bot run azure -- 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.

[GitHub] [hudi] weimingdiit commented on pull request #7362: [HUDI-5315] The record size is dynamically estimated when the table i…

2023-02-13 Thread via GitHub
weimingdiit commented on PR #7362: URL: https://github.com/apache/hudi/pull/7362#issuecomment-1427769776 @hudi-bot run azure -- 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.

[GitHub] [hudi] weimingdiit commented on pull request #7362: [HUDI-5315] The record size is dynamically estimated when the table i…

2023-02-01 Thread via GitHub
weimingdiit commented on PR #7362: URL: https://github.com/apache/hudi/pull/7362#issuecomment-1413253602 @hudi-bot run azure -- 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.

[GitHub] [hudi] weimingdiit commented on pull request #7362: [HUDI-5315] The record size is dynamically estimated when the table i…

2023-01-14 Thread GitBox
weimingdiit commented on PR #7362: URL: https://github.com/apache/hudi/pull/7362#issuecomment-1383022339 @hudi-bot run azure -- 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.