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

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2021/03/26 04:37:38 UTC

[GitHub] [incubator-pinot] sajjad-moradi commented on pull request #6613: Basic Auth for pinot-controller

sajjad-moradi commented on pull request #6613:
URL: https://github.com/apache/incubator-pinot/pull/6613#issuecomment-807929383


   > > lgtm other than a very minor comment. I have resolved all the comments of mine. Can you also work with @sajjad-moradi to resolve his comments as well?
   > > thanks for your patience.
   > 
   > Awesome. Thanks, Subbu. Yeah, so many changes with all the glue and tooling code.
   > 
   > Addressed your remaining comments, and waiting for @sajjad-moradi to circle back.
   
   LGTM.


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

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



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