Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Turbo module / Fabric support (New Architecture)? #6200

Answered by mikehardy
mfbx9da4 asked this question in Ideas
Discussion options

Is RN fire base planning to move to turbo module initialisation?

Thanks!

You must be logged in to vote

There are no current plans. I can say, having watched the progress of this as a member of the release-testers group - and seeing the pain it causes in for example the reanimated module, that it looks quite expensive to me, on the "module maintainership" side. So that's a negative: very high cost in new tech integration as it brings in new toolchains and requires module release cadence to sync with react-native release cadence - a significant new hurdle for all library consumers and a significant new dependency for release cycles.

(削除) If I understand correctly the sole benefit is that it is a performance optimization (削除ここまで) [Edit: this explains the benefits better, I'm still not sure they really appl...

Replies: 4 comments 21 replies

Comment options

There are no current plans. I can say, having watched the progress of this as a member of the release-testers group - and seeing the pain it causes in for example the reanimated module, that it looks quite expensive to me, on the "module maintainership" side. So that's a negative: very high cost in new tech integration as it brings in new toolchains and requires module release cadence to sync with react-native release cadence - a significant new hurdle for all library consumers and a significant new dependency for release cycles.

(削除) If I understand correctly the sole benefit is that it is a performance optimization (削除ここまで) [Edit: this explains the benefits better, I'm still not sure they really apply to react-native-firebase, but it is important to understand them:https://github.com/react-native-community/discussions-and-proposals/discussions/546#discussioncomment-4160151 ]

In our case, I'm not sure there is any actual benefit to it for performance improvements with the exception of firestore, which could benefit perhaps from better performance in data marshalling/unmarshalling. But then it is my understanding that most of the performance issues people have had with firestore are related to local cached data being accessed slowly due to underlying issues in the native SDKs.

So to me I see large costs + a lot of added complexity and I see very uncertain benefits. Thus I haven't put it on my short list of things todo personally

Side note is that our testing infrastructure requires the JSI interface and I'm not sure that works when Fabric is enabled. So if I understand correctly, this is blocked until our e2e infrastructure moves to a pure on-device solution - that's something we have in progress but it's not done yet.

You must be logged in to vote
18 replies
Comment options

@dmahajan980 i may be wrong. i am not able to find exact merged pr related to it. but i just read on https://blixtdev.com/react-native-0-72-whats-new/

update : please check this : reactwg/react-native-new-architecture#115

Comment options

That's fantastic! I have on my board to run new architecture compile tasks after seeing they were working on it for 0.72, I'll see if there are any gotchas, but in general this unblocks things here, fantastic news

Comment options

@mikehardy have there been any updates on this?

Comment options

No updates at the moment, but we're quite active here in the repo. We're nearly done with the work to adopt the firebase-js-sdk v9 apis which was a higher priority but also a lot of work as it is an ecosystem shift similar to new architecture

Eventually we will have no choice but to adopt new architecture here of course, but the work has not really begun yet

Comment options

Is Firebase v9 natively compatible with the new architecture on RN? I am running into issues loading Firebase through cocoa pods as well as using the react-native-firebase lib. I know you mentioned this,
#6368 (comment)

So I wasn't sure if that was just for rn-firebase or for the Firebase cocoapod as well

Answer selected by mfbx9da4
Comment options

Any change on this? With new architecture moving closer to becoming the defacto standard, is it still not yet planned?

You must be logged in to vote
3 replies
Comment options

#7688
It seems that New Architecture is now supported!

Comment options

@bglgwyng can you please confirm it is? According to the main expo docs:

https://docs.expo.dev/guides/new-architecture/#known-issues-in-third-party-libraries

it's not :(

Comment options

I tested upgrading my app to new architecture but haven't tested all the firebase features yet. I think you'd better try new architecture + firebase yourself.

Comment options

I'm assmuming this has been solved by #7688, is it correct?

You must be logged in to vote
0 replies
Comment options

New architecture compatibility was announced by @brentvatne on this tweet https://x.com/notbrent/status/1832108444578672709
Thank you @russellwheatley & invertase team

You must be logged in to vote
0 replies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

AltStyle によって変換されたページ (->オリジナル) /