activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From franz1981 <>
Subject [GitHub] activemq-artemis issue #1576: ARTEMIS-1447 JDBC NodeManager to support JDBC ...
Date Thu, 12 Oct 2017 05:45:03 GMT
Github user franz1981 commented on the issue:
    @mtaylor wdyt?
    > One thing regarding the SQL Provider.. you use the same class for all the statements.
Perhaps needs to be refactored...
    > So, if you for now just avoid constants in your statements.. and isolate the statements
well with clear docs.. perhaps we can refactor this in better terms. If you pass in arguments
to your statements directly from your internal enumerations it would be easier to understand
what's going on.


View raw message