[GitHub] [pinot] xiangfu0 commented on pull request #11620: Refactor BasicAuthUtils from pinot-core to pinot-common

["xiangfu0 (via GitHub)" <gi...@apache.org>]

xiangfu0 commented on PR #11620:
URL: https://github.com/apache/pinot/pull/11620#issuecomment-1725032777

   > Seems we do have some scalar functions defined in pinot-core. Is it okay to not include them?
   
   It's ok if we exclude entire `pinot-core`, as the scalar functions won't even be discovered.
   The previous issue is that the client wants to register those scaler functions from `pinot-core` but won't be able to find the dependencies.
   


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

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