jerrypeng commented on pull request #9348:
URL: https://github.com/apache/pulsar/pull/9348#issuecomment-769960599


   @eolivelli thanks for chiming in!
   
   > If you do not have feedback that something went wrong you are going to 
have garbage and you do not know.
   
   This is not entirely true.  If an error occurs, it is logged.
   
   The reason for this change:
   
   1) Currently, all cleanup operations of external resources used by a Pulsar 
Function is best effort.  There is no guarantee a resource like subscription or 
table will be cleanup.  We make a best effort but there is no guarantee.  Thus, 
this change does not really change any guarantees from that perspective.  We 
will be looking into ways to improve that in the future.
   
   2) Since our current model is best effort, there is no need to wait for the 
table deletion operation to complete.  Even though this is not an issue for 
Pulsar Functions, the state store client has some issues that we are 
investigating with cause it to block indefinitely. Thus, another reason to not 
to wait
   
   
   
   


----------------------------------------------------------------
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.

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


Reply via email to