You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@couchdb.apache.org by "nickva (via GitHub)" <gi...@apache.org> on 2023/09/18 16:29:24 UTC

[GitHub] [couchdb] nickva commented on pull request #4766: Add `smoosh:status/1` to HTTP API

nickva commented on PR #4766:
URL: https://github.com/apache/couchdb/pull/4766#issuecomment-1723898346

   I wonder if it's worth leaving room in the API for additional status which is not channel related? That is, move the channel status under a "channels" sub-field?:
   
   ```
   {"channels": [{"upgraded_views": {...}, ]}
   ```
   
   Smoosh has at least two other "status" pieces of information that may seem interesting to expose for debugging:
     * The overall `paused` state indicating if jobs should be paused or pausing based on the configured time window and the  strict setting.
     * If persistence is enabled, show the last time the state had been persisted, or if it was initialized from a persisted state, the timestamp of that state?
   
   Not saying we should add these right off the bat, but if we would them at any point in the future, make it possible to do so without completely changing the status response format.


-- 
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: notifications-unsubscribe@couchdb.apache.org

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