apache / lucene-jira-archive

Jira archive for Apache Lucene
https://lucene.apache.org/
2 stars 6 forks source link

Should we regenerate another full export? #104

Closed mikemccand closed 2 years ago

mikemccand commented 2 years ago

I know this is a hassle and adds delay, but, we have fixed a number of issues since the last full test export.

And e.g. @msokolov's first comment on this current iteration is just such an example.

If we take another iteration we can bring fresh eyes on the latest changes?

mocobeta commented 2 years ago

Yes - I plan another full import. I don't think we need to walk through the complete migration steps written in #7 again, but at least we can pick the most critical parts - conversion and import. Also we may want to iterate the issue id remapping part (LUCENE-XXXX to #NN) at least for the part of issues.

mocobeta commented 2 years ago

All issues were successfully imported to https://github.com/mocobeta/forks-migration-test-2/issues with the latest main. Issue link remapping (LUCENE-XXXX -> #NN) is ongoing.

mocobeta commented 2 years ago

Done. https://github.com/mocobeta/forks-migration-test-2/issues

Generally it looks fine. I noticed some minor glitches like #109 and #110. I'll address them (if it's easy and safe to fix). I think we can open an INFRA issue for the migration next week.

mocobeta commented 2 years ago

Before opening INFRA issues, I'll try to address issues I found in the latest migration test and run another full import.

mikemccand commented 2 years ago

Thanks @mocobeta.

mocobeta commented 2 years ago

Hi, here is the latest migration result. All the recent improvements (e.g., #128, #131, #136, etc.) were applied. https://github.com/mocobeta/forks-migration-test-3/issues

mocobeta commented 2 years ago

I checked several issues from new to old. I see no degradations from the previous one.

If you are okay with that, the actual migration will be the same as this except for the issue reporter and assignees (the issue reporter should be an ASF bot account and assignees should be correctly set).

mikemccand commented 2 years ago

+1, let's close this, yay!! Thank you @mocobeta!

mocobeta commented 2 years ago

Thank you, please open reports if you notice anything. I think we still have some time for small improvements.