blankensteiner commented on PR #207:
URL: https://github.com/apache/pulsar-dotpulsar/pull/207#issuecomment-2011678421

   Hi @smbecker and thank you for the PR :-)
   In regards to "bool HasReachedEndOfTopic();", this is a state and could be 
checked that way.
   In regards to "ValueTask<bool> HasMessageAvailable(CancellationToken 
cancellationToken = default);" I'm not sure why that is needed and why it is 
async? If you want to call "HasMessageAvailable" to make sure that Receive is 
not blocked, then that can not be guaranteed, because HasMessageAvailable might 
return true, but before you get to call Receive the Reader/Consumer can have 
lost the connection or faulted. Are that you really need something like this: 
   bool TryReceive(out Message message)?


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to