loopback-component-migrate icon indicating copy to clipboard operation
loopback-component-migrate copied to clipboard

Published 20 hours ago • verified publisher icon fullcube

MigrationMap unused?

Open jouke opened this issue 8 years ago • 2 comments

While working on the pull request I just submitted, I noticed that the MigrationMap model seems unused. Apart from the index.js defining the model and the files that define the model, there is absolutely no reference to it.

Can it be removed? I'd rather not have unused tables in my database... :)

Cheers,

Jouke

jouke avatar Aug 22 '16 11:08 jouke

I also agree with @jouke, MigrationMap seems unused and even it's unit tests are useless. Please let us know if you have not implemented it for a specific reason and it can be removed so we can submit a pull request for that.

Thanks for this useful component and I really appreciate any help you can provide.

alireza-ahmadi avatar Mar 04 '17 21:03 alireza-ahmadi

Hi, The original purpose of the MigrationMap collection was to enable you to store mappings between Object Ids pre and post migration which can be useful if you are migrating data from one system to another where Object Ids are being changed along the way. Arguably this is only useful for certain types of migrations and there are no useful tests for it.

I would consider a pull request that either adds a setting to make the creation of this model optional, or remove it completely from the component.

mrfelton avatar Mar 05 '17 13:03 mrfelton