tuhaihe commented on code in PR #55:
URL: https://github.com/apache/cloudberry-pxf/pull/55#discussion_r2771999324
##########
README.md:
##########
@@ -78,43 +74,35 @@ To build PXF, you must have:
go install github.com/onsi/ginkgo/ginkgo@latest
```
-5. cURL (7.29 or later):
-
- To install cURL devel package on CentOS 7, `sudo yum install
libcurl-devel`.
-
- Note that CentOS 6 provides an older, unsupported version of cURL (7.19).
You should install a newer version from source if you are on CentOS 6.
-
-### How to Build PXF
+### Build PXF
PXF uses Makefiles to build its components. PXF server component uses Gradle
that is wrapped into the Makefile for convenience.
```bash
-cd ~/workspace/pxf
+cd ~/pxf
-# Compile & Test PXF
+# Compile PXF
make
-
-# Only run unit tests
-make test
```
-### How to Install PXF
+### Install PXF
-To install PXF, first make sure that the user has sufficient permissions in
the `$GPHOME` and `$PXF_HOME` directories to perform the installation. It's
recommended to change ownership to match the installing user. For example, when
installing PXF as user `gpadmin` under `/usr/local/cloudberrydb`:
+To install PXF, first make sure that the user has sufficient permissions in
the `$GPHOME` and `$PXF_HOME` directories to perform the installation. It's
recommended to change ownership to match the installing user. For example, when
installing PXF as user `gpadmin` under `/usr/local/cloudberry-db`:
```bash
-export GPHOME=/usr/local/cloudberrydb
+mkdir -p /usr/local/pxf
+export GPHOME=/usr/local/cloudberry-db
export PXF_HOME=/usr/local/pxf
export PXF_BASE=${HOME}/pxf-base
chown -R gpadmin:gpadmin "${GPHOME}" "${PXF_HOME}"
Review Comment:
Good catch. I changed it to `source
/usr/local/cloudberry-db/cloudberry-env.sh` in commit
https://github.com/apache/cloudberry-pxf/pull/55/commits/5a5f5cc0ea52d149dcea07520902e9846dbf1797.
Does it look good?
--
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]
