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

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        17h 42m 1 Ted Dunning 06/Oct/11 19:26
        Resolved Resolved Closed Closed
        125d 19h 33m 1 Sean Owen 09/Feb/12 14:00
        Sean Owen made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        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
        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.
        Ted Dunning made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.6 [ 12316364 ]
        Resolution Fixed [ 1 ]
        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.
        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.
        Ted Dunning made changes -
        Field Original Value New Value
        Attachment MAHOUT-828.patch [ 12497932 ]
        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 created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development