[ https://issues.jboss.org/browse/TEIIDDES-1879?page=com.atlassian.jira.plu... ] Barry LaFond commented on TEIIDDES-1879: ---------------------------------------- {code} Thought I'd get started on this one since it required an Admin API change by adding a new method. I tested locally by changing the 8.4.x ExecutionAdmin.refreshTranslators() method to use the new admin.getTranslatorPropertyDefinitions() in place of admin.getTemplatePropertyDefinitions(). It returned different properties that appeared to be correct. So question is: How do we make our 8.4.x ExecutionAdmin. call the old method for 8.4 and 8.5 versions of Teiid, but switch to new method when version >= 8.6 ? Did we jump the gun in merging the 8.6 source? Barry {code} {code} * Does the new method not work correctly for 8.[4|5] versions of teiid? * If not, why not? * If not, is it not being backported to those versions? * At the end of my current work, there will be only 1 ExecutionAdmin class per major version. I was hoping for a single ExecutionAdmin class from 8+ with an extra v7 class hanging around due to differences in protocol. (Might be optimistic as teiid version 9 could be completely different!) * For such little differences like this, we have no choice but to detect the version of the server, which can be done using getServer().getServiceVersion(). ITeiidServerVersion has methods such as isGreatherThan / is LessThan etc... * This is a last resort!! The proliferation of checking the teiid server version has so far been checked to a few instances. We don't want to have it spreading like a plague of locusts over the code. PGR {code} -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira