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/10/28 08:03:00 UTC
[GitHub] [druid] gianm opened a new pull request, #13273: RTR: Dedupe items in getKnownTasks.
gianm opened a new pull request, #13273:
URL: https://github.com/apache/druid/pull/13273
Fixes a problem where the tasks API in OverlordResource would complain about duplicate keys in the map it's building.
--
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
[GitHub] [druid] gianm merged pull request #13273: RTR: Dedupe items in getKnownTasks.
Posted by GitBox <gi...@apache.org>.
gianm merged PR #13273:
URL: https://github.com/apache/druid/pull/13273
--
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
[GitHub] [druid] gianm commented on pull request #13273: RTR: Dedupe items in getKnownTasks.
Posted by GitBox <gi...@apache.org>.
gianm commented on PR #13273:
URL: https://github.com/apache/druid/pull/13273#issuecomment-1294717103
> So I guess there would be duplicates if say a task moves from "PENDING" to "RUNNING" and the running set is updated but the the pending set is still not updated?
The sets can be updated while `getKnownTasks` is running, so it's possible for the same task to be in both `pendingTasks` and `runningTasks`, if it moves in the time between reading those two sets.
--
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