Github user robertamarton commented on a diff in the pull request:

    https://github.com/apache/incubator-trafodion/pull/141#discussion_r42791691
  
    --- Diff: core/sqf/sql/scripts/install_local_hadoop ---
    @@ -1534,7 +1534,7 @@ fi
     
     cd $MY_SW_ROOT
     
    -if [ ! -r $YARN_HOME/lib/native/libhdfs.so ]; then
    +if [ -d $MY_LOCAL_SW_DIST && ! -r $YARN_HOME/lib/native/libhdfs.so ]; then
       echo "------------------------------------------------------------"
       echo "-- WARNING: libhdfs.so is not present on this system. Please"
       echo "--          build it, otherwise Trafodion will not compile."
    --- End diff --
    
    The libhdfs library is needed at both build and run time.  The way we have 
things setup, we should be creating this directory at build time, and copying 
the file at run time.  There may be a situation where somehow the library goes 
missing.  To avoid confusion to people who are trying out the product and 
should follow the build instructions from those who actually maintain this 
script, I will leave the message in but only display it if VERBOSE is enabled.  
During debugging, if someone is having issue, then this information maybe 
useful.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to