Bigtop
  1. Bigtop
  2. BIGTOP-504

provide a functionality for detecting libjvm.so in bigtop-utils

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.4.0
    • Fix Version/s: 0.6.0
    • Component/s: general
    • Labels:
      None

      Description

      Currently, for hdfs fuse we have code lile this in the launcher script:

      if [ "\${LD_LIBRARY_PATH}" = "" ]; then
      export LD_LIBRARY_PATH=/usr/lib
      for f in \`find \${JAVA_HOME}/jre/lib -name client -prune -o -name libjvm.so -exec dirname {} \;\`; do
      export LD_LIBRARY_PATH=\$f:\${LD_LIBRARY_PATH}
      done
      fi
      

      It would be nice to provide this as a general purpose detection code in bigtop-utils.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        322d 3h 1m 1 Roman Shaposhnik 19/Feb/13 02:51
        Resolved Resolved Closed Closed
        122d 20h 58m 1 Roman Shaposhnik 22/Jun/13 00:49
        Roman Shaposhnik made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Roman Shaposhnik made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Roman Shaposhnik added a comment -

        +1 and committed!

        Show
        Roman Shaposhnik added a comment - +1 and committed!
        Hide
        Sean Mackrory added a comment -

        I've also made sure this doesn't suffer from any previous problems in bigtop-utils that I'm aware of, such as JAVA_HOME being a symlink.

        Show
        Sean Mackrory added a comment - I've also made sure this doesn't suffer from any previous problems in bigtop-utils that I'm aware of, such as JAVA_HOME being a symlink.
        Sean Mackrory made changes -
        Attachment BIGTOP-504.1.patch [ 12569282 ]
        Hide
        Sean Mackrory added a comment -

        Moved the `find` command into a new script called bigtop-detect-javalibs (similar to bigtop-detect-javahome - which means the invocation will be further simplified if and when we resolve BIGTOP-463).

        I've also made this a bit more flexible so this can also serve for other components that may needed it. Components list the libraries they need to link against dynamically in JAVA_NATIVE_LIBS (in this case just libjvm.so), and the required paths are returned in JAVA_NATIVE_PATH - which can be prepended to LD_LIBRARY_PATH.

        Show
        Sean Mackrory added a comment - Moved the `find` command into a new script called bigtop-detect-javalibs (similar to bigtop-detect-javahome - which means the invocation will be further simplified if and when we resolve BIGTOP-463 ). I've also made this a bit more flexible so this can also serve for other components that may needed it. Components list the libraries they need to link against dynamically in JAVA_NATIVE_LIBS (in this case just libjvm.so), and the required paths are returned in JAVA_NATIVE_PATH - which can be prepended to LD_LIBRARY_PATH.
        Sean Mackrory made changes -
        Assignee Bruno Mahé [ bmahe ] Sean Mackrory [ mackrorysd ]
        Hide
        Bruno Mahé added a comment -

        Sure, go ahead.

        Show
        Bruno Mahé added a comment - Sure, go ahead.
        Hide
        Sean Mackrory added a comment -

        Bruno - I didn't see this JIRA before, and I recently started working on BIGTOP-845 and some other changes to bigtop-utils. Have you already worked on this, or do you mind if I take it over? Just didn't want to step on any toes or duplicate work.

        Show
        Sean Mackrory added a comment - Bruno - I didn't see this JIRA before, and I recently started working on BIGTOP-845 and some other changes to bigtop-utils. Have you already worked on this, or do you mind if I take it over? Just didn't want to step on any toes or duplicate work.
        Roman Shaposhnik made changes -
        Fix Version/s 0.6.0 [ 12323895 ]
        Fix Version/s 0.5.0 [ 12321865 ]
        Roman Shaposhnik made changes -
        Fix Version/s 0.5.0 [ 12321865 ]
        Fix Version/s 0.4.0 [ 12318889 ]
        Gavin made changes -
        Field Original Value New Value
        Workflow no-reopen-closed, patch-avail [ 12660771 ] patch-available, re-open possible [ 12665794 ]
        Roman Shaposhnik created issue -

          People

          • Assignee:
            Sean Mackrory
            Reporter:
            Roman Shaposhnik
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development