Skip to content

Fix: Fixed an issue where the app could hang while updating relative date columns - #18852

Merged
yair100 merged 1 commit into
mainfrom
ya/fix-date-display-hang
Aug 20, 2026
Merged

Fix: Fixed an issue where the app could hang while updating relative date columns#18852
yair100 merged 1 commit into
mainfrom
ya/fix-date-display-hang

Conversation

@yair100

@yair100 yair100 commented Aug 20, 2026

Copy link
Copy Markdown
Member

Resolved / Related Issues

To prevent extra work, all changes to the Files codebase must link to an approved issue marked as Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.

Steps used to test these changes

Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.

  1. Opened Files ...
  2. ...

@yair100 yair100 closed this Aug 20, 2026
@yair100
yair100 deleted the ya/fix-date-display-hang branch August 20, 2026 04:33
@yair100
yair100 restored the ya/fix-date-display-hang branch August 20, 2026 04:56
@yair100 yair100 reopened this Aug 20, 2026
@yair100
yair100 requested a review from hez2010 August 20, 2026 04:57
@hez2010

hez2010 commented Aug 20, 2026

Copy link
Copy Markdown
Member

Actually I think we can keep the AsParallel().ForAll().

But instead of queuing and awaiting the dispatcher work there, we can do the queue work only dispatcherQueue.TryEnqueue(Microsoft.UI.Dispatching.DispatcherQueuePriority.Low, item.UpdateReal).

Haven't yet verified whether this would work though.

Update: this doesn't work. The current fix is the right move.

Also link #18848?

@yair100 yair100 added the ready to merge Pull requests that are approved and ready to merge label Aug 20, 2026
@yair100
yair100 merged commit dec94fe into main Aug 20, 2026
18 checks passed
@yair100
yair100 deleted the ya/fix-date-display-hang branch August 20, 2026 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Pull requests that are approved and ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Keeps freezing

2 participants