Hey all, There are two PRs on applying the changes as per this discussion, requiring review and approval:
1. https://github.com/apache/cloudberry-devops-release/pull/19 * Add PAX build for daily PR tests * Also, update the dependencies for PAX under the Rocky Linux 8 build image. 2. https://github.com/apache/cloudberry/pull/1081 * After PR#19 above is applied, we can rebase #1081 to update the configure option for PAX. Welcome to have your review and feedback, thanks! Best, Dianjin Wang On Thu, May 15, 2025 at 11:03 AM Zhang Mingli <avamin...@apache.org> wrote: > > Hi, > > On 2025/05/14 06:01:23 Dianjin Wang wrote: > > Will it require a major refactoring effort to install the PAX using > > the EXTENSION method? > > PAX codes have already organized to contrib/pax_storage as a extension, when > build CBDB with PAX, it will go to the contrib/pax_storage dir and exec make/ > make install. > So it will not be so hard. > > We are close to the new release; if so, hope we > > can evolve in the future release. > > Never rush to do a crucial thing, make things right. > In fact, these should be discussed in previous PAX's Pull Requests. > > > > > Best, > > Dianjin Wang > > > > On Wed, May 14, 2025 at 11:34 AM Zhang Mingli <avamin...@apache.org> wrote: > > > > > > > > > Hi, Leonid > > > > > > On 2025/05/10 14:23:29 Leonid Borchuk wrote: > > > > Hi, all > > > > > > > > I really like the PostgreSQL approach - configure && make && make > > > > install. > > > > > > Me too. > > > > > > > And usually there are no additional packages or builds required. > > > > Postgresql > > > > seems to be compiled everywhere - even on coffee machine. It would be > > > > great > > > > to see the same for cloudberry. Since PAX is quite complex feature it > > > > would > > > > be better to have a special option --enable-pax. > > > > > > Agree. > > > > > > Unlike AO, PAX, as a contrib module, I think we should avoid adding > > > --enable-pax or --disable-pax flags entirely. > > > > > > Instead, we should treat it as a standard PostgreSQL contrib module. That > > > means: > > > It gets built(compiled and installed) only if users explicitly install > > > it from its directory and follows the usual extension workflow (CREATE > > > EXTENSION) > > > > > > Extensions should follow the rule of Postgres, users who need PAX can > > > enable it the same way they would any other extension. > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: dev-unsubscr...@cloudberry.apache.org > > > For additional commands, e-mail: dev-h...@cloudberry.apache.org > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@cloudberry.apache.org > > For additional commands, e-mail: dev-h...@cloudberry.apache.org > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@cloudberry.apache.org > For additional commands, e-mail: dev-h...@cloudberry.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@cloudberry.apache.org For additional commands, e-mail: dev-h...@cloudberry.apache.org
