my-ship-it commented on issue #17: URL: https://github.com/apache/cloudberry-pxf/issues/17#issuecomment-3797704591
> I spent some time fixing cloudberry/pxf_fdw. The code [MisterRaindrop@647a85e](https://github.com/MisterRaindrop/cloudberry-pxf/commit/647a85e4ffcda77113ae94eb071eafb1d3452c53) has been re-adapted to Cloudberry, and all regression tests for pxf_fdw are now passing. > > In my opinion, pxf_fdw is just a plugin and should ideally not be placed together with kernel code. This would allow for more flexibility in version control. > > I would appreciate more suggestions. From my perspective, I agree with you, and it would be more reasonable to put the pxf fdw code together with the pxf plugin, which is easy to future maintenance and upgrades? -- 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]
