Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Labels:
    • Environment:
      Mac OS X 10.6, Groovy 1.7.6, Java 1.6.0

      Description

      kitty> domains
      domains
      Domains Detected:
      0: JMImplementation
      1: Users
      2: com.sun.management
      3: Catalina
      4: java.lang
      5: java.util.logging
      kitty> setdomain Catalina
      setdomain Catalina
      Setting the domain to Catalina...
      Cannot invoke method length() on null object

      1. kitty-12.patch
        0.4 kB
        Alessandro Novarini

        Activity

        Hide
        Alessandro Novarini added a comment -

        The fastest way to fix it is to correct the typo

        Class org.apache.kitty.client.Client
        Method setDomain

        if (it.equals(_domain))

        { domain = _domain mBeanPath = [] <<<<<<<<< should be mBeansPath }
        Show
        Alessandro Novarini added a comment - The fastest way to fix it is to correct the typo Class org.apache.kitty.client.Client Method setDomain if (it.equals(_domain)) { domain = _domain mBeanPath = [] <<<<<<<<< should be mBeansPath }
        Hide
        Alessandro Novarini added a comment -

        This is just the fix, but the code could be refactored, right after the availability of some unit test

        Show
        Alessandro Novarini added a comment - This is just the fix, but the code could be refactored, right after the availability of some unit test
        Hide
        Matthew Sacks added a comment -

        Patch applied, fix works.

        kitty> connect 127.0.0.1 9000
        connect 127.0.0.1 9000
        connecting to 127.0.0.1 at port 9000....
        the url is service:jmx:rmi:///jndi/rmi://127.0.0.1:9000/jmxrmi
        Successfully connected to host
        kitty> help
        help
        COMMANDS

        connect <host> <port> - Connect to the remote host
        disconnect - Disconnect from remote host
        cd <path> - Change the current mbean path
        exit - Quit the application and return to the command prompt
        get <attribute> - Get an attribute from the remote host
        set <attribute> <value> - Set an attribute on the remote host
        setdomain <domain> - Set the domain in the current session
        domains - List all available MBeanServer domainsinvoke <operation> <parameters...> - Invoke an operation on the remote host
        ls - List all objects and attributes in current mbean path
        pwd - print working directory of mbean path

        kitty> domains
        domains
        Domains Detected:
        0: JMImplementation
        1: Users
        2: com.sun.management
        3: Catalina
        4: java.lang
        5: java.util.logging
        kitty> setdomain Catalina
        setdomain Catalina
        Setting the domain to Catalina...
        The domain is set to Catalina

        Show
        Matthew Sacks added a comment - Patch applied, fix works. kitty> connect 127.0.0.1 9000 connect 127.0.0.1 9000 connecting to 127.0.0.1 at port 9000.... the url is service:jmx:rmi:///jndi/rmi://127.0.0.1:9000/jmxrmi Successfully connected to host kitty> help help COMMANDS connect <host> <port> - Connect to the remote host disconnect - Disconnect from remote host cd <path> - Change the current mbean path exit - Quit the application and return to the command prompt get <attribute> - Get an attribute from the remote host set <attribute> <value> - Set an attribute on the remote host setdomain <domain> - Set the domain in the current session domains - List all available MBeanServer domainsinvoke <operation> <parameters...> - Invoke an operation on the remote host ls - List all objects and attributes in current mbean path pwd - print working directory of mbean path kitty> domains domains Domains Detected: 0: JMImplementation 1: Users 2: com.sun.management 3: Catalina 4: java.lang 5: java.util.logging kitty> setdomain Catalina setdomain Catalina Setting the domain to Catalina... The domain is set to Catalina
        Hide
        Matthew Sacks added a comment -

        Patch applied and confirmed fix.

        Show
        Matthew Sacks added a comment - Patch applied and confirmed fix.
        Hide
        Pid added a comment -

        Please reopen if this is not complete.

        Show
        Pid added a comment - Please reopen if this is not complete.

          People

          • Assignee:
            Matthew Sacks
            Reporter:
            Matthew Sacks
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development