Re: [PR] [HUDI-7788] Fixing exception handling in AverageRecordSizeUtils [hudi]

2024-05-24 Thread via GitHub


yihua merged PR #11290:
URL: https://github.com/apache/hudi/pull/11290


-- 
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



Re: [PR] [HUDI-7788] Fixing exception handling in AverageRecordSizeUtils [hudi]

2024-05-24 Thread via GitHub


yihua commented on PR #11290:
URL: https://github.com/apache/hudi/pull/11290#issuecomment-2130376301

   Azure CI is green.
   https://github.com/apache/hudi/assets/2497195/d1207e62-c420-40a6-81ab-a26c6b835fd5;>
   


-- 
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



Re: [PR] [HUDI-7788] Fixing exception handling in AverageRecordSizeUtils [hudi]

2024-05-24 Thread via GitHub


yihua merged PR #11289:
URL: https://github.com/apache/hudi/pull/11289


-- 
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



Re: [PR] [HUDI-7788] Fixing exception handling in AverageRecordSizeUtils [hudi]

2024-05-24 Thread via GitHub


yihua commented on PR #11289:
URL: https://github.com/apache/hudi/pull/11289#issuecomment-2130375906

   Azure CI is green.
   https://github.com/apache/hudi/assets/2497195/42f68992-cc98-4456-bd39-9248c0fef4dc;>
   


-- 
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



Re: [PR] [HUDI-7788] Fixing exception handling in AverageRecordSizeUtils [hudi]

2024-05-24 Thread via GitHub


hudi-bot commented on PR #11290:
URL: https://github.com/apache/hudi/pull/11290#issuecomment-2130151684

   
   ## CI report:
   
   * 853d6b5d25c5724250bb8d9d03bef07aa5ed73bb UNKNOWN
   
   
   Bot commands
 @hudi-bot supports the following commands:
   
- `@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



Re: [PR] [HUDI-7788] Fixing exception handling in AverageRecordSizeUtils [hudi]

2024-05-24 Thread via GitHub


hudi-bot commented on PR #11289:
URL: https://github.com/apache/hudi/pull/11289#issuecomment-2130151416

   
   ## CI report:
   
   * 1fa21effcf407ec954ae805c2604e6de2239ae22 UNKNOWN
   
   
   Bot commands
 @hudi-bot supports the following commands:
   
- `@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



[PR] [HUDI-7788] Fixing exception handling in AverageRecordSizeUtils [hudi]

2024-05-24 Thread via GitHub


yihua opened a new pull request, #11290:
URL: https://github.com/apache/hudi/pull/11290

   ### Change Logs
   
   PR targeting master https://github.com/apache/hudi/pull/11289
   This PR targets `branch-0.x` with the same changes.
   
   This PR makes the exception handling in `AverageRecordSizeUtils` more robust 
by catching the `Throwable` instead of `IOException`.
   
   ### Impact
   
   Robust exception handling in `AverageRecordSizeUtils`
   
   ### Risk level
   
   none
   
   ### Documentation Update
   
   none
   
   ### Contributor's checklist
   
   - [ ] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [ ] Change Logs and Impact were stated clearly
   - [ ] Adequate tests were added if applicable
   - [ ] CI passed
   


-- 
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



[PR] [HUDI-7788] Fixing exception handling in AverageRecordSizeUtils [hudi]

2024-05-24 Thread via GitHub


yihua opened a new pull request, #11289:
URL: https://github.com/apache/hudi/pull/11289

   ### Change Logs
   
   This PR makes the exception handling in `AverageRecordSizeUtils` more robust 
by catching the `Throwable` instead of `IOException`.
   
   ### Impact
   
   Robust exception handling in `AverageRecordSizeUtils`
   
   ### Risk level
   
   none
   
   ### Documentation Update
   
   none
   
   ### Contributor's checklist
   
   - [ ] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [ ] Change Logs and Impact were stated clearly
   - [ ] Adequate tests were added if applicable
   - [ ] CI passed
   


-- 
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