[Crash] ASDeallocQueue (15): Fatal error: Executing on background thread. Please use `MainScheduler.instance.schedule` to schedule work on main thread.

[mongris]

With following code:

class MyView: ASDisplayNode {
  let data = BehaviorRelay<Section>

  override func didLoad() {
        super.didLoad()

        data.asDriver()
            .map({ [$0] })
            .drive(collectionView.rx.items(dataSource: dataSource))
            .disposed(by: disposeBag)
    }
}

The app will be crash when dismissed view controller because ASDisplayNode was dealloc
in background thread.

[mongris]

It can be fix with workaround in ASDelegateProxy.swift from line 43 - 58:

                    // ensure running unregisterDelegate.dispose() on main thread
                    DispatchQueue.main.async {
                        switch event {
                        case .error(let error):
                            bindingErrorToInterface(error)
                            unregisterDelegate.dispose()
                        case .completed:
                            unregisterDelegate.dispose()
                        default:
                            break
                        }
                    }
                }

            return Disposables.create { [weak object] in
                // ensure running unregisterDelegate.dispose() on main thread
                DispatchQueue.main.async {
                    subscription.dispose()
                    object?.layoutIfNeeded()
                    unregisterDelegate.dispose()
                }
            }

[fumito-ito]

@mongris thank you for your contributions. Could you upload sample project to reproduce this problem ?

[mongris]

RxASDataSourcesCrashSample.zip

@fumito-ito I have attached a sample project to repo this issue.

repo steps:
After starting the app, navigate to collectionView then back to main view with navigation left button.

[fumito-ito]

thanks @mongris ! I will check it later.

[mongris]

afd0a51 fixed it.

[JustinM1]

@fumito-ito can we get a release bump containing this fix?

[fumito-ito]

@JustinM1 thanks for your request. We are reviewing next release at #24. It will be released in a few days.