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/07 13:58:01 UTC

[GitHub] [druid] abhishekagarwal87 commented on pull request #13045: fix bug in /status/properties filtering

abhishekagarwal87 commented on PR #13045:
URL: https://github.com/apache/druid/pull/13045#issuecomment-1239428601

   Great catch @capistrant. LGTM. can you also add an assertion that the size of returned properties is less than `injector.getInstance(Properties.class).size()`? That way, such an issue won't happen again. 


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