ecerulm commented on PR #23504:
URL: https://github.com/apache/airflow/pull/23504#issuecomment-1119490076

   
   >  Maybe it's just a matter of phrasing and what it means is that the 
resource just no longer exists (if it ever existed). But that's very 
missleading.
   
   Yes, I think it's misleading too. That's why I thought I could sort by 
resource version, but no, resource versions are just opaque ids. 
   
   > And then that means there are no other way than deal with [the bookmark 
events](https://github.com/kubernetes-client/python/issues/1729) to implement 
this correctly, right?
   
   I don't think the bookmark thing solves it either as 
[stated](https://kubernetes.io/docs/reference/using-api/api-concepts/#watch-bookmarks):
 
   
   > but you shouldn't assume bookmarks are returned at any specific interval, 
nor can clients assume that the API server will send **any BOOKMARK event even 
when requested**
   
   I have another PR #23521 which just resets to a fresh watch if there is any 
error. It will probably get some old (already processed) events but I don't 
think that breaks anything. 


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