[GitHub] [flink] WencongLiu commented on pull request #22341: [FLINK-27204] Refract FileSystemJobResultStore to execute I/O operations on the ioExecutor

2023-07-15 Thread via GitHub


WencongLiu commented on PR #22341:
URL: https://github.com/apache/flink/pull/22341#issuecomment-1636989057

   Thanks for your careful review  @XComp. Sorry for the first round of pull 
request because it's a bit rough. Please take a look when you have time.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [flink] WencongLiu commented on pull request #22341: [FLINK-27204] Refract FileSystemJobResultStore to execute I/O operations on the ioExecutor

2023-06-05 Thread via GitHub


WencongLiu commented on PR #22341:
URL: https://github.com/apache/flink/pull/22341#issuecomment-1576632397

   @XComp I have made a round of changes. Please take a look at it when you 
have time. 


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [flink] WencongLiu commented on pull request #22341: [FLINK-27204] Refract FileSystemJobResultStore to execute I/O operations on the ioExecutor

2023-05-19 Thread via GitHub


WencongLiu commented on PR #22341:
URL: https://github.com/apache/flink/pull/22341#issuecomment-1554191565

   @XComp I have made a round of changes. Please take a look at it when you 
have time. 


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [flink] WencongLiu commented on pull request #22341: [FLINK-27204] Refract FileSystemJobResultStore to execute I/O operations on the ioExecutor

2023-05-17 Thread via GitHub


WencongLiu commented on PR #22341:
URL: https://github.com/apache/flink/pull/22341#issuecomment-1550846620

   @flinkbot 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.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [flink] WencongLiu commented on pull request #22341: [FLINK-27204] Refract FileSystemJobResultStore to execute I/O operations on the ioExecutor

2023-05-10 Thread via GitHub


WencongLiu commented on PR #22341:
URL: https://github.com/apache/flink/pull/22341#issuecomment-1543301331

   @XComp Thanks for your reply! 
   I only have a small questions. What should I define the return types of
   
   `void createDirtyResult(JobResultEntry) `
   `void markResultAsClean(JobID)`
   
   Currently the invokers of these two methods need sync behaviors. If we give 
it a Completable,and the invokers must use future.get(). From this point, 
refracting return types of these two methods is a little meaningless.
   


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [flink] WencongLiu commented on pull request #22341: [FLINK-27204] Refract FileSystemJobResultStore to execute I/O operations on the ioExecutor

2023-04-12 Thread via GitHub


WencongLiu commented on PR #22341:
URL: https://github.com/apache/flink/pull/22341#issuecomment-1505221567

   @flinkbot 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.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [flink] WencongLiu commented on pull request #22341: [FLINK-27204] Refract FileSystemJobResultStore to execute I/O operations on the ioExecutor

2023-04-08 Thread via GitHub


WencongLiu commented on PR #22341:
URL: https://github.com/apache/flink/pull/22341#issuecomment-1500819380

   @flinkbot 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.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [flink] WencongLiu commented on pull request #22341: [FLINK-27204] Refract FileSystemJobResultStore to execute I/O operations on the ioExecutor

2023-04-06 Thread via GitHub


WencongLiu commented on PR #22341:
URL: https://github.com/apache/flink/pull/22341#issuecomment-1498895544

   @flinkbot 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.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org