You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pegasus.apache.org by "empiredan (via GitHub)" <gi...@apache.org> on 2025/11/24 09:48:58 UTC

[PR] refactor: introduce host_port::resolve() use it instead of dns_resolver::instance()::resolve_address() [incubator-pegasus]

empiredan opened a new pull request, #2326:
URL: https://github.com/apache/incubator-pegasus/pull/2326

   (no comment)


-- 
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: dev-unsubscribe@pegasus.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org


Re: [PR] refactor: introduce `host_port::resolve()` and use it instead of `dns_resolver::instance()::resolve_address()` [incubator-pegasus]

Posted by "Samunroyu (via GitHub)" <gi...@apache.org>.
Samunroyu commented on PR #2326:
URL: https://github.com/apache/incubator-pegasus/pull/2326#issuecomment-3681380276

   > > @empiredan pls check the failed test.
   > 
   > Translation:
   > 
   > @Samunroyu The `throttle_test.test` fails at times due to its own implementation, not because of this PR. It will be tracked in this issue and handled separately: #2282.
   
   I see, LGTM


-- 
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: dev-unsubscribe@pegasus.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org


Re: [PR] refactor: introduce `host_port::resolve()` and use it instead of `dns_resolver::instance()::resolve_address()` [incubator-pegasus]

Posted by "Samunroyu (via GitHub)" <gi...@apache.org>.
Samunroyu commented on PR #2326:
URL: https://github.com/apache/incubator-pegasus/pull/2326#issuecomment-3645724166

   @empiredan pls check the failed test.


-- 
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: dev-unsubscribe@pegasus.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org


Re: [PR] refactor: introduce `host_port::resolve()` and use it instead of `dns_resolver::instance()::resolve_address()` [incubator-pegasus]

Posted by "empiredan (via GitHub)" <gi...@apache.org>.
empiredan merged PR #2326:
URL: https://github.com/apache/incubator-pegasus/pull/2326


-- 
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: dev-unsubscribe@pegasus.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org


Re: [PR] refactor: introduce `host_port::resolve()` and use it instead of `dns_resolver::instance()::resolve_address()` [incubator-pegasus]

Posted by "empiredan (via GitHub)" <gi...@apache.org>.
empiredan commented on PR #2326:
URL: https://github.com/apache/incubator-pegasus/pull/2326#issuecomment-3646117236

   > @empiredan pls check the failed test.
   
   Translation:
   
   @Samunroyu The `throttle_test.test` fails at times due to its own implementation, not because of this PR. It will be tracked in this issue and handled separately: https://github.com/apache/incubator-pegasus/issues/2282.
   


-- 
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: dev-unsubscribe@pegasus.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org