[forge-commits] [forge/core] 785c2f: FORGE-2055 Create a cache for UICommands

Lincoln.Baxter at lists.jboss.org Lincoln.Baxter at lists.jboss.org
Thu Oct 2 17:31:21 EDT 2014


  Branch: refs/heads/master
  Home:   https://github.com/forge/core
  Commit: 785c2f4e6f51e0d1d73604f57cdd9ed49fe73ced
      https://github.com/forge/core/commit/785c2f4e6f51e0d1d73604f57cdd9ed49fe73ced
  Author: Vineet Reynolds <Vineet.Reynolds at gmail.com>
  Date:   2014-10-03 (Fri, 03 Oct 2014)

  Changed paths:
    M shell/impl/src/main/java/org/jboss/forge/addon/shell/aesh/ForgeCommandRegistry.java
    M ui/api/src/main/java/org/jboss/forge/addon/ui/command/CommandFactory.java
    M ui/impl/src/main/java/org/jboss/forge/addon/ui/impl/command/CommandFactoryImpl.java

  Log Message:
  -----------
  FORGE-2055 Create a cache for UICommands

A cache for UICommands is introduced for any operation that
does not mutate the state of the UICommand, for instance,
displaying the command names as list, in the Shell or in an IDE.

This commit also introduces a new API to ensure that new UICommand
instances are created when commands are executed. Cached UICommands
are not used at this point to ensure that cached UIInput values are
not used.


  Commit: 214163e29923e406f1bab6539f5e9cea9786fb11
      https://github.com/forge/core/commit/214163e29923e406f1bab6539f5e9cea9786fb11
  Author: Lincoln Baxter, III <lincolnbaxter at gmail.com>
  Date:   2014-10-02 (Thu, 02 Oct 2014)

  Changed paths:
    M shell/impl/src/main/java/org/jboss/forge/addon/shell/aesh/ForgeCommandRegistry.java
    M ui/api/src/main/java/org/jboss/forge/addon/ui/command/CommandFactory.java
    M ui/impl/src/main/java/org/jboss/forge/addon/ui/impl/command/CommandFactoryImpl.java

  Log Message:
  -----------
  Merge pull request #505 from VineetReynolds/command-cache-perf-fix

Command cache performance fix


Compare: https://github.com/forge/core/compare/4db120b05527...214163e29923


More information about the forge-commits mailing list