Re: [opennms-devel] Database backwards-compatibility change in OpenNMS 17

[Seth L. <se...@op...>]

Hi Dustin,

Just write a migration for the Linkd table removals. Yeah, rollbacks
aren't necessary for that. There was no logical way to write rollbacks
for the column removals either.

-- Seth


On 5/21/15 8:44 AM, Dustin Frisch wrote:
> Is there anything to do for us if we are going to remove the linkd
> tables?
>
> We do not have to build rollback statements for removing tables as we
> are loosing rollback capability anyways, right?
>
>
> Seth Leger <se...@op...> wrote:
>
>> Hi everyone,
>>
>> I wanted to give everyone a heads-up that there will be a Liquibase
>> migration in OpenNMS 17 that will remove some columns and triggers
>> from the database in the ipInterface, ifServices, and outages tables.
>>
>> These columns and triggers were in place to preserve backwards
>> compatibility with older JDBC code that used a node/ipAddr/service
>> composite key instead of the simpler numeric primary keys that we use
>> with Hibernate. However, since Capsd has been removed and a lot of
>> refactoring has been done to remove old JDBC code and replace it with
>> Hibernate, we can now get rid of these database columns without
>> causing problems. This will increase performance because of the
>> reduced number of database calls due to the removal of the triggers.
>>
>> If you are working in the 'develop' branch and upgrade to
>> 17.0.0-SNAPSHOT after the Liquibase migration is committed, the
>> migration will remove the database columns. If you want to go back to
>> developing on an older version or a branch that does not contain the
>> database changes, you will need to have A BACKUP OF YOUR DATABASE that
>> you can restore that has the columns and triggers (or you will need to
>> install from scratch). Otherwise, the older version of OpenNMS will
>> undoubtedly crash because of queries that are expecting those columns
>> in the database.
>>
>> The changes are not yet committed to 'develop' for inclusion in 17
>> but I expect them to pass unit tests today and be ready to merge by
>> tomorrow.
>>
>> Heads up!!
>> Seth Leger
>> The OpenNMS Group
>>
>>
>> ------------------------------------------------------------------------------
>> One dashboard for servers and applications across
>> Physical-Virtual-Cloud Widest out-of-the-box monitoring support with
>> 50+ applications Performance metrics, stats and reports that give you
>> Actionable Insights Deep dive visibility with transaction tracing
>> using APM Insight.
>> http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
>> _______________________________________________ Please read the
>> OpenNMS Mailing List FAQ:
>> http://www.opennms.org/index.php/Mailing_List_FAQ
>>
>> opennms-devel mailing list
>>
>> To *unsubscribe* or change your subscription options, see the bottom
>> of this page:
>> https://lists.sourceforge.net/lists/listinfo/opennms-devel