[GitHub] [pulsar] heesung-sn commented on pull request #20044: [improve][client] Add isInterrupted func on TableView

["heesung-sn (via GitHub)" <gi...@apache.org>]

heesung-sn commented on PR #20044:
URL: https://github.com/apache/pulsar/pull/20044#issuecomment-1500813083

   
   1. retry forever upon failures.
   2. throw RuntimeException from the data accessor apis, if already interrupted.(get, entrySet, keySet)
   3. expose table view state(e.g isInterrupted)
   
   It appears that we have the above options.
   
   If tableview is supposed to guarantee the liveness, I think we can go to the option 1 instead.


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

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