+
Skip to content

Adding merge_into/2 function #4

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

Merged
merged 1 commit into from
Mar 2, 2020
Merged

Adding merge_into/2 function #4

merged 1 commit into from
Mar 2, 2020

Conversation

farhadi
Copy link
Contributor

@farhadi farhadi commented Feb 24, 2020

merge_into/2 is like merge/1 except that instead of creating a new blex struct it merges blexes into the given blex struct as second argument.

PS: some changes are because of running mix format

@gyson
Copy link
Owner

gyson commented Feb 25, 2020

Thanks for the contribution!

Just curious, do you have a concrete use case for this new merge_into/2 API ?

@farhadi
Copy link
Contributor Author

farhadi commented Feb 25, 2020

I have a use case where a blex struct is shared between a lot of processes and at some occasions I want to update this struct with some other blex binaries I receive from other nodes in the cluster.
If I don't merge them to this existing struct, I have to send a newly created struct to all processes.

@gyson gyson merged commit 062548d into gyson:master Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载