You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hop.apache.org by GitBox <gi...@apache.org> on 2022/02/01 10:43:16 UTC

[GitHub] [hop] mattcasters commented on pull request #1305: HOP-3713 Fixed BlockingRowSet high cpu, support dynamic timeout when get row from InputRowSet

mattcasters commented on pull request #1305:
URL: https://github.com/apache/hop/pull/1305#issuecomment-1026704390


   Wonderful PR @shlxue, great work & happy new year. I'll do further testing and a performance review later this week.  It would be interesting to make configuration options HOP_ROWSET_GET_MAX_TIMEOUT and TIMEOUT_GET_MAX_MILLIS part of the LocalPipelineRunConfiguration metadata so that it's exposed more conveniently.  However, we can do that in a separate PR.


-- 
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-unsubscribe@hop.apache.org

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