Mahout
  1. Mahout
  2. MAHOUT-828

bin/mahout should only print classpath on request, not all the time

    Details

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

      Description

      This is a trivial change, but it changes the user experience. Anybody care to keep the current (very) verbose output?

        Activity

        Ted Dunning created issue -
        Hide
        Ted Dunning added a comment -

        I will commit this in a day or so once comments have come in. This patch was accidentally attached to MAHOUT-829 as well. I have fixed that.

        Show
        Ted Dunning added a comment - I will commit this in a day or so once comments have come in. This patch was accidentally attached to MAHOUT-829 as well. I have fixed that.
        Ted Dunning made changes -
        Field Original Value New Value
        Attachment MAHOUT-828.patch [ 12497932 ]
        Hide
        Lance Norskog added a comment -

        Yes, please stop printing the classpath. Also, please export the classpath to the app in the CLASSPATH environment variable, not on the command line. On some unixes, there is a maximum length in the command line printed by 'ps -f'. The classpath uses all of the space, and I can't see which mahout job is running.

        Show
        Lance Norskog added a comment - Yes, please stop printing the classpath. Also, please export the classpath to the app in the CLASSPATH environment variable, not on the command line. On some unixes, there is a maximum length in the command line printed by 'ps -f'. The classpath uses all of the space, and I can't see which mahout job is running.
        Hide
        Ted Dunning added a comment -

        Committed this due to non-controversial nature.

        The

        mahout classpath

        command will now print the classpath.

        Also, the classpath is passed in the environment. I expect that the command will still be too long to read, but it is a reasonable thing to try.

        Show
        Ted Dunning added a comment - Committed this due to non-controversial nature. The mahout classpath command will now print the classpath. Also, the classpath is passed in the environment. I expect that the command will still be too long to read, but it is a reasonable thing to try.
        Ted Dunning made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.6 [ 12316364 ]
        Resolution Fixed [ 1 ]
        Hide
        Sean Owen added a comment -

        Ted I don't think you actually committed this to SVN. I'll apply it.

        Show
        Sean Owen added a comment - Ted I don't think you actually committed this to SVN. I'll apply it.
        Hide
        Hudson added a comment -
        Show
        Hudson added a comment - Integrated in Mahout-Quality #1106 (See https://builds.apache.org/job/Mahout-Quality/1106/ ) MAHOUT-828 srowen : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1186717 Files : /mahout/trunk/bin/mahout
        Sean Owen made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Ted Dunning
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development