You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2022/09/22 08:18:44 UTC

[GitHub] [druid] rohangarg commented on pull request #13015: Enable using SHARED lock for append ingestion jobs

rohangarg commented on PR #13015:
URL: https://github.com/apache/druid/pull/13015#issuecomment-1254686270

   Closing this since there are multiple test failures after enabling the context flag (not necessarily related to bugs, but to different configurations in tests which would need tweaking). 
   Since MSQ already uses SHARED locks for `INSERTS` and EXCLUSIVE locks for `REPLACE`, we should be leveraging that capability instead of spending time on this.
   It can be re-opened if we think otherwise in the future.


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

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org