New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[docs] updated HDFS guide #1890
Conversation
docs/Installation-Guide.rst
Outdated
@@ -569,6 +569,8 @@ On Linux HDFS version of LightGBM can be built using **CMake** and **gcc** or ** | |||
git clone --recursive https://github.com/Microsoft/LightGBM ; cd LightGBM | |||
mkdir build ; cd build | |||
cmake -DUSE_HDFS=ON .. | |||
# if you have installed HDFS to a customized location, you should specify paths to HDFS headers (hdfs.h) and library (libhdfs.so) like the following: | |||
# cmake -DUSE_HDFS=ON -DHDFS_LIB_DIR=/usr/lib/hadoop-hdfs/lib -DHDFS_INCLUDE_DIR=/usr/lib/hadoop-hdfs/include/ .. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe ask users to provide HDFS_LIB
directly instead of HDFS_LIB_DIR
?
https://github.com/Microsoft/LightGBM/blob/f55e37d02670374085d523357f0032ac544b6ed2/CMakeLists.txt#L109
docs/Installation-Guide.rst
Outdated
@@ -553,7 +553,7 @@ Refer to `GPU Docker folder <https://github.com/Microsoft/LightGBM/tree/master/d | |||
Build HDFS Version | |||
~~~~~~~~~~~~~~~~~~ | |||
|
|||
**Note**: Installation process of HDFS version is untested. | |||
HDFS version of LightGBM was tested on CDH cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ebernhardson Can you please provide more details (i.e., version) to help users setup environment in which LightGBM will work fine?
@ebernhardson Please review. |
gently ping @ebernhardson |
Updated based on the comments from #2046.
I suppose that after merging this, #1555 can be closed. WDYT? |
Sure |
Closed #1555. |
Refer to #1555 (comment).