Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrating from Mage1 -> Mage2 can be optimized #1000

Closed
SchumacherFM opened this issue Jan 26, 2015 · 1 comment
Closed

Migrating from Mage1 -> Mage2 can be optimized #1000

SchumacherFM opened this issue Jan 26, 2015 · 1 comment

Comments

@SchumacherFM
Copy link
Member

I have written SQL database migration tool (not yet open source) which runs pretty well.

The tools does a bunch of fixes and writes all executed SQL scripts into core_resource tables column version. Column data_version is null.

After my migration tools has finished the setup scripts in the modules data folder needs to be run to apply some Magento2 stuff to your Magento1 database. I call the script:

root/setup $ php index.php update

Due to existing data from Magento1 the data updates will fail. I have to disable the data update for modules:

            'log' => '1.6.0.0',
            'checkout' => '2.0.0',
            'directory' => '2.0.0',
            'sales' => '2.0.0',
            'reports' => '2.0.0',
            'review' => '2.0.0',
            'tax' => '2.0.0',

and also commented out in files:

  • Customer/data/customer_setup/data-install-2.0.0.php Lines 12 - 130
  • Catalog/data/catalog_setup/data-install-2.0.0.php Lines 12-44, 94-127 (maybe some more?)
  • CatalogInventory/data/cataloginventory_setup/data-install-2.0.0.php Lines 10-14

to be able to run the index.php update tool.

I suggest the following: Either for all inserted data use MySQL INSERT IGNORE to silently fail on existing entries OR separate the logic in the data install script between adding data and migrating data (then it is easier to pick the correct install/update script).

AFAIK: there is also a migration script missing in the dev/tools/Magento/Tools/Migration folder which converts the e.g. backend_model to the new class names in the EAV tables.

@TexanHogman
Copy link

@SchumacherFM to satisfy several requirements we are taking a different approach to support migration to M2.

The existing feature of update scripts, requires the site to be in maintenance mode and because some operations could take a significant amount time doing conversions we were not comfortable relying on this for our M2 data migration needs.

In M2 we are supporting migration using a new tool and workflow that allows extremely fast data migration using mostly native sql commands. The process will be to setup a fresh Magento 2 instance and run the tool to pull data from the M1 database. This allows validation in a staging environment and also a final (very quick) catchup phase therefore minimizing the downtime for the site. The solution uses a mapping file between source and target versions of Magento. The core team will provide this mapping files for several versions of CE and EE and the community can pitch in too to provide support for other versions.

Additionally, as you have seen we have collapsed all M1 update scripts for M2 this allows us to start fresh with M2 but of course limits the update scripts support versions. Since the incremental upgrade did not solve performance requirements (site downtime) we decided to start fresh for M2.

We understand this is an important merchant/si first experience with M2 and therefore we are ensuring this is fast, extensible and predictable.

magento-team pushed a commit that referenced this issue Apr 5, 2017
- MAGETWO-65119 Create/update automated functional tests
- MAGETWO-66178 Update marketing information, order page and order list with new labels
- MAGETWO-66240 Add tags for tests that require single flow run
- MAGETWO-63642 Update Order Status Based on Signifyd Guarantee Statuses
magento-team pushed a commit that referenced this issue Dec 13, 2017
…or! #1000

 - Merge Pull Request magento-engcom/magento2ce#1000 from RomaKis/magento2:8204
 - Merged commits:
   1. 9e85ead
   2. 97e43c1
magento-team pushed a commit that referenced this issue Dec 13, 2017
Merge pull request #1014 from magento-engcom/2.2-develop-prs

[EngCom] Public Pull Requests - 2.2-develop

Public Pull Requests
 - #1012 10814: Attribute repository resets sourceModel for new attributes. by @nmalevanec
- #1006 #12285: The option false for mobile device don't work in product -
view page gallery by @p-bystritsky
- #983 #12259: Save and Duplicated product not working by @p-bystritsky
- #1000 8204: catalog:images:resize = getimagesize(): Read error! by @RomaKis
#12633 Magento Connect no longer exist by @miguelbalparda

Fixed Public Issues
 - #10814 Attribute repository resets sourceModel for new attributes
 - #12285 The option false for mobile device don't work in product view page gallery
 - #12490 I can't disable full screen gallery on mobile on magento 2.2.1
 - #12259 Save and Duplicated product not working
 - #8204 catalog:images:resize = getimagesize(): Read error! in vendor/magento/module-catalog/Model/Product/Image.php on line 410 if an image is 0 bytes
 - #12632 Magento Connect no longer exist
VitaliyBoyko pushed a commit to VitaliyBoyko/magento2 that referenced this issue Jun 22, 2018
…-increment-decimal

Fix issue 774 qty increment decimal
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants