You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@couchdb.apache.org by GitBox <gi...@apache.org> on 2022/07/14 18:11:15 UTC

[GitHub] [couchdb] chewbranca commented on pull request #4106: 4101 add io priority

chewbranca commented on PR #4106:
URL: https://github.com/apache/couchdb/pull/4106#issuecomment-1184752330

   Thanks @nickva, I forgot to push out the changes cleaning up those remaining pdict puts from the earlier commit.
   
   One thing that occurred to me, setting pdict values is _always_ a valid thing in respect to dependency hierarchies, whereas using `ioq` module functions requires `ioq` to be declared as a dependency. We already do that for the `couch` app https://github.com/apache/couchdb/blob/main/src/couch/src/couch.app.src#L46 but I'll double check that the other places we're doing so also include the `ioq` app in their dependency hierarchy. 


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