You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@inlong.apache.org by GitBox <gi...@apache.org> on 2021/09/17 12:15:53 UTC

[GitHub] [incubator-inlong] TszKitLo40 opened a new issue #1568: C++ SDK cant not return the whether the partition has been registered correctly

TszKitLo40 opened a new issue #1568:
URL: https://github.com/apache/incubator-inlong/issues/1568


   <!--
   Thanks for contributing to InLong!
   Please explain your issue precisely, and if possible provide a reproducer
   snippet (this helps resolve issues much quicker).
   -->
   
   #### Expected behavior
   
   Tell us what should happen
   C++ SDK cant not return the whether the partition has been registered correctly
   #### Actual behavior
   
   Tell us what happens instead
   
   https://github.com/apache/incubator-inlong/blob/4b2223269da92aacd0cec8d40da4d624dc42ca3e/inlong-tubemq/tubemq-client-twins/tubemq-client-cpp/src/rmt_data_cache.cc#L458
   
   In the function `IsPartitionFirstReg`, if `part_reg_booked_` can't find the partition, it will add the partition to the map and return true. And if the function is called second time, it will also return true because the status has been set to true.
   #### Steps to reproduce
   
   How can we reproduce the issue
   
   #### System configuration
   **InLong version**: 0.11.0
   
   
   


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

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

[GitHub] [incubator-inlong] dockerzhang closed issue #1568: C++ SDK cant not return the whether the partition has been registered correctly

Posted by GitBox <gi...@apache.org>.
dockerzhang closed issue #1568:
URL: https://github.com/apache/incubator-inlong/issues/1568


   


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

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

[GitHub] [incubator-inlong] gosonzhang commented on issue #1568: C++ SDK cant not return the whether the partition has been registered correctly

Posted by GitBox <gi...@apache.org>.
gosonzhang commented on issue #1568:
URL: https://github.com/apache/incubator-inlong/issues/1568#issuecomment-922172309


   @TszKitLo40, yes, you are right, thanks!
   
   I'll fix 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: commits-unsubscribe@inlong.apache.org

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