-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
2984 optimizing requestid #2985
Conversation
Modified the compilation of the artifacts in order to generate a unique identifier (md5 text) for Request type nodes. writeRelayGeneratedFile: eliminate the switch, the node is always of type Request because inside 'if (generatedNode.kind === RelayConcreteNode.REQUEST) {' https://github.com/facebook/relay/blob/master/packages/relay-compiler/codegen/writeRelayGeneratedFile.js#L98
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jstejada has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
this will make persisting the store to cause less memory more info here morrys/react-relay-offline#34 (comment) |
@morrys this PR is already imported and we're looking at how to roll it out internally |
#2984
Modified the compilation of the artifacts in order to generate a unique identifier (md5 text) for the 'Request' nodes.
modified writeRelayGeneratedFile:
eliminate the switch, the node is always of type Request because inside 'if (generatedNode.kind === RelayConcreteNode.REQUEST) {'
https://github.com/facebook/relay/blob/master/packages/relay-compiler/codegen/writeRelayGeneratedFile.js#L98