glassfish
  1. glassfish
  2. GLASSFISH-2485

add target host and port arguments to appclient command-line

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 9.1pe
    • Fix Version/s: not determined
    • Component/s: standalone_client
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      2,485

      Description

      Running an appclient component on a different machine than the server is an increasingly common
      scenario. For those developers who choose to invoke the client container using the appclient
      command rather than webstart, it would be good to add some optional target host / port arguments
      to the appclient program.

      That would represent a significant ease-of-use improvement over having to define a sun-acc.xml file,
      set the host/port in the file, and then pass that to the command-line.

      e.g. appclient -host host1 -port 3700 ...

        Activity

        Hide
        Tim Quinn added a comment -

        What about using a single option, perhaps something like -server, that would
        accept a comma list of values of the form host:port? The user could then
        specify a boot-failover list using -server host1:3700,host2:33700. This would
        fully replicate that feature of the target spec in the sun-acc.xml file.

        And it would be a bit simpler even for specifying a single host and port, saving
        a bit of typing for -server host1:3700 (as opposed to -host host1 -port 3700).

        Is "server" the best name for the option?

        Show
        Tim Quinn added a comment - What about using a single option, perhaps something like -server, that would accept a comma list of values of the form host:port? The user could then specify a boot-failover list using -server host1:3700,host2:33700. This would fully replicate that feature of the target spec in the sun-acc.xml file. And it would be a bit simpler even for specifying a single host and port, saving a bit of typing for -server host1:3700 (as opposed to -host host1 -port 3700). Is "server" the best name for the option?
        Hide
        ksak added a comment -

        That sounds ok as long as the simple case of one target is well documented/understood since that's the
        90% scenario. We don't want people thinking the option only applies to some cluster scenario.

        If we combine the host/port syntax , it would also be good to make the port portion optional since 3700 is
        the default. Something like a comma separated list of [<host> | <host>:<port>]. (Seems reasonable to
        require host if port is specified.)

        That way I can just do appclient -server host1 ... and get the default of 3700 for port.

        -server seems fine to me The only issue is whether people will confuse it with the java -server option
        that controls the client vs. server mode for the JVM itself.

        Show
        ksak added a comment - That sounds ok as long as the simple case of one target is well documented/understood since that's the 90% scenario. We don't want people thinking the option only applies to some cluster scenario. If we combine the host/port syntax , it would also be good to make the port portion optional since 3700 is the default. Something like a comma separated list of [<host> | <host>:<port>] . (Seems reasonable to require host if port is specified.) That way I can just do appclient -server host1 ... and get the default of 3700 for port. -server seems fine to me The only issue is whether people will confuse it with the java -server option that controls the client vs. server mode for the JVM itself.
        Hide
        Tom Mueller added a comment -

        Bulk change to set fix version to "not determined" where the issue is open but the value is for a released version.

        Show
        Tom Mueller added a comment - Bulk change to set fix version to "not determined" where the issue is open but the value is for a released version.

          People

          • Assignee:
            Tim Quinn
            Reporter:
            ksak
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: