tuhaihe commented on PR #1383: URL: https://github.com/apache/cloudberry/pull/1383#issuecomment-3409728750
> > Would like to have more voices from the core PAX developers on this. cc @gfphoenix78 @jiaqizho @gongxun0928 > > At least, it's not good to place the `yyjson` separately at the top of the dir. > > The reason why `dependency/yyjson` is separated and stored instead of being placed under PAX is because not only PAX depends on `yyjson`. Does the core of cloud also rely on `yyjson`? > Thanks @jiaqizho for your reply. I remember our private conversation on this topic. ❤️ Now, from the open-source side, I believe we can move the `yyjson` to the correct place. If some downstream vendors refer to it as a dependency, the downstream should have a patch for it or clone it separately. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
