Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 295382

[meeting] Migrate Meeting 2024-11-21 1400Z

$
0
0

Core migration issues

Next video meeting 2025-01-02 (tentative)

Hello all, it’s time for the biweekly migration subsystem meeting. The meeting will take place in slack in various threads. This meeting:
➤ Is for core migrate maintainers and developers and anybody else in the community with an interest in migrations
➤ Usually happens every second Thursday and alternates between 1400 and 2100 UTC.
➤ Is done on the #migration channel in Drupal Slack (see www.drupal.org/slack for information).
➤ Happens in threads, which you can follow to be notified of new replies even if you don’t comment in the thread. You may also join the meeting later and participate asynchronously!
➤ Has a public agenda anyone can add to. #3488926: [meeting] Migrate Meeting 2024-11-21 1400Z See the parent issue for an idea of the typical agenda.
Transcript will be exported and posted to the agenda issue. For anonymous comments, start with a 👤 emoji. To take a comment or thread off the record, start with a 🚫 emoji.

0️⃣ Who is here today?

anmolgoyal74Hi :wave:
mikelutz (he/him)Hi
dinarcon:wave:
Dan DavisHello!
godotislate:wave:
benjifisherI posted in the channel, but maybe I should have said it here:

I have to multi-task for the next 1/2 hour. :disappointed:

1️⃣ What should we talk about today? Suggest topics here and I will add threads. I will also check for comments on the issue for today's meeting.

dinarconI'd like to revisit some discussions related to performance from early in the year. One particular issue is[#3463800] but a general thread on performance would be nice (edited)
dinarconAnd somewhat related to performance... what are some tips and tricks for profiling slow/long running migrations? Maybe this can be included in the performance thread? (edited)
dinarconhttps://www.drupal.org/project/drupal/issues/2630732 is marked as needs review. @larowlan already reviewed it last time it was set to RTBC and asked to close some threads in the MR. Some of the open threads are about suggestions in code style. Others seem to have an impact on how the new feature would work. Shall we talk about how to move the issue forward? Maybe continue working on the MR suffices.
benjifisherI think that AUTO_INCREMENT Alter is appropriate for the 6️⃣ Announcements thread. (edited)

2️⃣ Action items. To be added later.

3️⃣ Statistics

Migrate Initiative Meeting351 total, 0 Fixed, 1 RTBC (Major), 3 NR (1 Major)
Migrate Initiative MeetingGoogle sheet for recording stats: https://docs.google.com/spreadsheets/d/1o0Rjlc1vnnLP5bM5P-SMMyGzqn7258hi...

4️⃣ Comment in this thread if you are looking for ways to contribute. Give us some idea of what you would like to do: documentation, code review, testing, project management, ...

anmolgoyal74I am looking to contribute. I can help in documentation, testing , development
benjifisherI do not think anyone is making regular documentation updates.The top-level guide is https://www.drupal.org/docs/drupal-apis/migrate-api.
benjifisherThe page on debugging is more up-to-date than I had feared: https://www.drupal.org/docs/drupal-apis/migrate-api/debugging-migrations. But it does not mention that (as of Drupal 10.3/11.0) Drupal core provides migration messages in the admin UI. You might start with that.
benjifisherChange record: https://www.drupal.org/node/3371203

5️⃣ Previous minutes.

Migrate Initiative Meetinghttps://www.drupal.org/project/drupal/issues/3483191 (edited)
smustgraveThere are number of old meeting tickets going back to 2020, some of them needing transcripts how best to handle those?
benjifisherThere is a Chrome plugin we use to scrape comments from Slack so that we can post them to the issue. It works with Chromium on Linux. https://github.com/mdlutz24/drupal-meeting-parser
smustgraveOh I mean I think it’s missing Youtube transcripts
smustgraveIf I come across slack ones I’ll see if someone wants to update.  Personally scrolling 5 years worth of threads sounds miserable
Dan DavisI just posted the transcript for 2024-11-07 -#3483191: [meeting] Migrate Meeting 2024-11-07 2100Z#comment-15867520

6️⃣ Announcements

dinarconAs part of writing the new D7 -> D10 migration series I released a new module to help with preventing entity ID conflicts by altering the AUTO_INCREMENT values of tables related to content entities. https://www.drupal.org/project/auto_increment_alter General feedback would be appreciated. Also wondering what people think about extending it to other drivers like psql and sqlite
dinarconAlso, sharing the latest article in the D7 to D10 upgrade series https://www.tag1consulting.com/blog/migrating-d7-d10-avoiding-entity-id-...
sameraligreat read thanks @dinarcon

7️⃣ Performance

Migrate Initiative Meetingwhat are some tips and tricks for profiling slow/long running migrations?
Migrate Initiative Meeting(Original Request) https://drupal.slack.com/archives/C226VLXBP/p1732199709426659?thread_ts=... (edited)
dinarconI have been trying to get batch size and highwater to work without much success. Tested the patch in this issue, but ended up with an infinite loop. In another thread, @mikelutz recommends having custom source plugins. Wondering if he, or someone else, could expand on this.
dinarconIn a recent project, I found that \Drush\Drupal\Migrate\MigrateExecutable::handleMissingSourceRows takes a significant amount of time to finish processing when there is a large number of records (in the 100 thousands range). In particular, this check was taking a lot of time to process...
      if (!in_array($mapSourceId, $this->allSourceIdValues))
    

First time I notice this. Still wrapping my head around this. Wondering if someone else has noticed this to be a bottleneck in slow/long running migrations.

dinarconThe above is in fact part of \Drush\Drupal\Migrate\MigrateExecutable. Not sure if it is wise to overwrite that at all. Although I think that migrate_tools does that to some extent.
dinarconIn case someone is experiencing something similar... I had not realized that \Drush\Drupal\Migrate\MigrateExecutable::handleMissingSourceRows is in fact part of Drush runner, not Drupal core code. We tested the migration runner from migrate_tools module and this issue with migrations halting at the end of the import was gone. So, switching runners helped in our case. Still trying to better understand what \Drush\Drupal\Migrate\MigrateExecutable::handleMissingSourceRows does and how it can be improved.
benjifisherThis is the thread from two weeks ago.
benjifisherDo you want to start a new thread for today's meeting?
dinarconYes, I should have posted earlier :sweat_smile: not sure if a new thread is necessary, but open to it.

8️⃣ Implement Entity::fields() for migration destinations

Migrate Initiative MeetingThis issue is back to NR after at least two "rerolls".
Migrate Initiative Meeting(Original Request) https://drupal.slack.com/archives/C226VLXBP/p1732200323887989?thread_ts=... (edited)
benjifisherIn my last review (Comment #166) I noticed (while closing threads) that one of the requested changes was either never made (despite comments that it had been made) or got lost in one of the rebases.
benjifisherSo I need to find time to do a complete review. It is on my to-do list.

9️⃣ Wrap-Up

Migrate Initiative MeetingThanks for coming all! See you in 2 weeks
benjifisherPlease consider the meeting open for at least the next 23 hours.
dinarconThanks for facilitating Benji.

Viewing all articles
Browse latest Browse all 295382

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>