You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "bogdandrutu (via GitHub)" <gi...@apache.org> on 2023/05/16 21:27:19 UTC

[GitHub] [thrift] bogdandrutu opened a new pull request, #2808: [Java]: Cleanup extend client logic for sync and async clients

bogdandrutu opened a new pull request, #2808:
URL: https://github.com/apache/thrift/pull/2808

   <!-- Explain the changes in the pull request below: -->
   
   Small cleanup for extend logic for both sync and async clients.
   
   <!--
     The Contributing Guide at:
     https://github.com/apache/thrift/blob/master/CONTRIBUTING.md
     has more details and tips for committing properly.
   -->
   


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

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

[GitHub] [thrift] ctubbsii commented on pull request #2808: [Java]: Cleanup extend client logic for sync and async clients

Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on PR #2808:
URL: https://github.com/apache/thrift/pull/2808#issuecomment-1558050214

   @bogdandrutu No. I can merge it. I was waiting to see if another person reviewed it, and thought maybe a second committer could merge after they approved it, but it's been open awhile, and is pretty trivial, so I'll go ahead and merge it.


-- 
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: notifications-unsubscribe@thrift.apache.org

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

[GitHub] [thrift] ctubbsii merged pull request #2808: [Java]: Cleanup extend client logic for sync and async clients

Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii merged PR #2808:
URL: https://github.com/apache/thrift/pull/2808


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

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

[GitHub] [thrift] bogdandrutu commented on pull request #2808: [Java]: Cleanup extend client logic for sync and async clients

Posted by "bogdandrutu (via GitHub)" <gi...@apache.org>.
bogdandrutu commented on PR #2808:
URL: https://github.com/apache/thrift/pull/2808#issuecomment-1558001549

   @ctubbsii anything else to do to merge this?


-- 
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: notifications-unsubscribe@thrift.apache.org

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