Never mind what I said, sorry about confusion. Looks like, based on the "base" result name I was deriving the replication name as we need both. Original commit is fine. Corrected now.

Also, there was a new name=value (resultset-cache-name=teiid-cache) added to the following:

/profile=ha/subsystem=teiid:add(async-thread-pool=teiid-async, distributed-cache-jgroups-stack=udp, resultset-cache-infinispan-container=teiid-cache, resultset-cache-name=teiid-cache, preparedplan-cache-infinispan-container=teiid-cache, policy-decider-module=org.jboss.teiid)

And I don't see this in standalone script, are they suppose to be different?

Tried with latest domain cli script committed to 8.4.x, and it still fails with:

[Server:server-one] JBAS014775: New missing/unsatisfied dependencies:
[Server:server-one] service jboss.infinispan.teiid-cache.resultset-repl-repl (missing) dependents: [service jboss.teiid.cache.resultset]
[Server:server-one]

server has an issue when the caches are assigned the cache names,not the container names. The following error is seen with current CLI scripts:

[Server:server-two] JBAS014775: New missing/unsatisfied dependencies:
[Server:server-two] service jboss.infinispan.preparedplan (missing) dependents: [service jboss.teiid.infinispan-pp-cache-factory]
[Server:server-two] service jboss.infinispan.preparedplan.preparedplan (missing) dependents: [service jboss.teiid.cache.prepared-plan]
[Server:server-two] service jboss.infinispan.resultset-repl (missing) dependents: [service jboss.teiid.infinispan-rs-cache-factory]
[Server:server-two] service jboss.infinispan.resultset-repl.resultset (missing) dependents: [service jboss.teiid.cache.resultset]
[Server:server-two] service jboss.infinispan.resultset-repl.resultset-repl (missing) dependents: [service jboss.teiid.cache.resultset]
[Server:server-two]
[Server:server-two] 11:06:28,817 ERROR [org.jboss.as] (Controller Boot Thread) JBAS015875: JBoss EAP 6.1.0.GA (AS 7.2.0.Final-redhat-8) started (with errors) in 7222ms - Started 196 of 338 services (13 services failed or missing dependencies, 128 services are passive or on-demand)

which do not appear when those changes are mode.

for HA add,

resultset-cache-name=resultset-repl

then we should be good

hmm.. good point may be not. Just the replication resultset cache name is not default in that case

Ramesh since we use those cache names by default in TeiidAdd, is there a reason that they need to be in the cli script as well?

Those are still in-correct. It should be

resultset-cache-infinispan-container=teiid-cache, resultset-cache-name=resultset, preparedplan-cache-infinispan-container=teiid-cache, preparedplan-cache-name= preparedplan

and for HA
resultset-cache-infinispan-container=teiid-cache, resultset-cache-name=resultset-repl, preparedplan-cache-infinispan-container=teiid-cache, preparedplan-cache-name= preparedplan

Pull submitted is wrong