Skip to content

Optimization batch 9: avoid detecting irrelevant renames#845

Closed
newren wants to merge 8 commits intogitgitgadget:temporary/ort-perf-batch-8from
newren:ort-perf-batch-9
Closed

Optimization batch 9: avoid detecting irrelevant renames#845
newren wants to merge 8 commits intogitgitgadget:temporary/ort-perf-batch-8from
newren:ort-perf-batch-9

Conversation

@newren
Copy link
Copy Markdown

@newren newren commented Jan 24, 2021

This series determines paths which meet special cases where detection
of renames is irrelevant, where the irrelevance is due to the fact
that the merge machinery will arrive at the same result regardless of
whether a rename is detected for any of those paths.

Changes since v2:

  • Added an extended comment about filtering order to the first patch, to address Stolee's question
  • Added a new testcase that will fail if the critical "if (content_relevant || location_relevant)" check only checks for one of those two or the intersection of those two, as suggested by Stolee
  • Fixed the other minor problems Stolee pointed out in his review (stray newline, wording of comment)

cc: Derrick Stolee dstolee@microsoft.com
cc: Jonathan Tan jonathantanmy@google.com
cc: Taylor Blau me@ttaylorr.com
cc: Junio C Hamano gitster@pobox.com
cc: Ævar Arnfjörð Bjarmason avarab@gmail.com
cc: Elijah Newren newren@gmail.com
cc: Derrick Stolee stolee@gmail.com

@newren newren force-pushed the temporary/ort-perf-batch-8 branch from 3bfbd40 to f3cda8d Compare January 28, 2021 00:16
@newren newren force-pushed the ort-perf-batch-9 branch 3 times, most recently from e4a8539 to 0a72a5e Compare February 3, 2021 05:32
@newren newren force-pushed the temporary/ort-perf-batch-8 branch from f3cda8d to c53920b Compare February 3, 2021 19:03
@newren newren force-pushed the ort-perf-batch-9 branch 2 times, most recently from 69d8e51 to d33d841 Compare February 3, 2021 20:33
@newren newren force-pushed the temporary/ort-perf-batch-8 branch from ac1fea5 to 4e99977 Compare February 9, 2021 10:33
@newren newren force-pushed the temporary/ort-perf-batch-8 branch from 4e99977 to d266fa7 Compare February 10, 2021 14:08
@newren newren force-pushed the ort-perf-batch-9 branch 2 times, most recently from a6e7c2e to ec57808 Compare February 10, 2021 16:35
@newren newren force-pushed the temporary/ort-perf-batch-8 branch from e546ae2 to e60bea9 Compare February 11, 2021 07:37
@newren newren force-pushed the temporary/ort-perf-batch-8 branch from e60bea9 to 81bcfc4 Compare February 12, 2021 17:29
@newren newren force-pushed the ort-perf-batch-9 branch 2 times, most recently from 4f3d6a7 to 0d3c472 Compare February 12, 2021 21:12
@newren newren force-pushed the temporary/ort-perf-batch-8 branch from 46774dc to 8b2286c Compare February 14, 2021 03:31
@newren newren force-pushed the ort-perf-batch-9 branch 2 times, most recently from 4077b17 to 65ea72d Compare February 14, 2021 03:39
@newren newren force-pushed the temporary/ort-perf-batch-8 branch 2 times, most recently from 805c101 to bfab7ff Compare February 25, 2021 01:21
@newren newren force-pushed the ort-perf-batch-9 branch 4 times, most recently from d40e692 to 8fed92b Compare February 27, 2021 06:17
@newren
Copy link
Copy Markdown
Author

newren commented Feb 28, 2021

/submit

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Feb 28, 2021

Submitted as pull.845.git.1614484707.gitgitgadget@gmail.com

To fetch this version into FETCH_HEAD:

git fetch https://github.com/gitgitgadget/git pr-845/newren/ort-perf-batch-9-v1

To fetch this version to local tag pr-845/newren/ort-perf-batch-9-v1:

git fetch --no-tags https://github.com/gitgitgadget/git tag pr-845/newren/ort-perf-batch-9-v1

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 22, 2021

This patch series was integrated into seen via git@217f038.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 22, 2021

This patch series was integrated into seen via git@dcdea95.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 22, 2021

This patch series was integrated into seen via git@c8f71b8.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 23, 2021

This patch series was integrated into seen via git@4c3ff01.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 23, 2021

There was a status update about the branch en/ort-perf-batch-9 on the Git mailing list:

The ort merge backend has been optimized by skipping irrelevant
renames.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 23, 2021

This patch series was integrated into seen via git@d419527.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 24, 2021

This patch series was integrated into seen via git@3d4cdfd.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 26, 2021

This patch series was integrated into seen via git@e21f66d.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 28, 2021

This patch series was integrated into seen via git@2571112.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 28, 2021

This patch series was integrated into seen via git@f53ac67.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 29, 2021

This patch series was integrated into seen via git@dacf9fb.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 30, 2021

This patch series was integrated into seen via git@2c6478c.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 30, 2021

This patch series was integrated into seen via git@40cd16d.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 31, 2021

There was a status update about the branch en/ort-perf-batch-9 on the Git mailing list:

The ort merge backend has been optimized by skipping irrelevant
renames.

Will merge to 'next'.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Mar 31, 2021

This patch series was integrated into seen via git@e02194b.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 2, 2021

This patch series was integrated into seen via git@187f96d.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 2, 2021

This patch series was integrated into next via git@cc5254a.

@gitgitgadget gitgitgadget bot added the next label Apr 2, 2021
@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 2, 2021

This patch series was integrated into seen via git@3341158.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 4, 2021

This patch series was integrated into seen via git@c512342.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 6, 2021

There was a status update about the branch en/ort-perf-batch-9 on the Git mailing list:

The ort merge backend has been optimized by skipping irrelevant
renames.

Will merge to 'master'.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 8, 2021

This patch series was integrated into seen via git@e7ceae4.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 8, 2021

This patch series was integrated into seen via git@bf979f8.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 8, 2021

This patch series was integrated into seen via git@1b31224.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 8, 2021

This patch series was integrated into next via git@1b31224.

@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 8, 2021

This patch series was integrated into master via git@1b31224.

@gitgitgadget gitgitgadget bot added the master label Apr 8, 2021
@gitgitgadget
Copy link
Copy Markdown

gitgitgadget bot commented Apr 8, 2021

Closed via 1b31224.

@gitgitgadget gitgitgadget bot closed this Apr 8, 2021
@newren newren deleted the ort-perf-batch-9 branch April 23, 2021 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant