theyenaman

theyenaman

Code Review Request - Toy cryptocurrency miner

I’m taking a class this semester that has me coding little projects in Elixir. Since this is the first time I’m working with this kind of a programming model, I’d really appreciate some help and guidance about how to structure stuff better, I’d really love to learn more about elixir and erlang.

Here’s a link to the project.

Please note that I’m not asking for help in completing this, the project deadline is past and it has been submitted for grading. I want your help to improve it and learn from possible design mistakes.

For reference this is a link to the original prompt

Specifically, my questions would be:

  1. Is there is a better way to structure my process tree, and how do I supervise this whole tree correctly?
  2. How do I make this OTP compliant (for the sake of learning about OTP, I’m not even sure if this is the right way to do this)

Any other feedback, comments are most welcome!

Here is an excerpt from the project README about the architecture:

Architecture

  • There is a single globally named process that manages all worker nodes and itself called the MiningServer which is a GenServer with the name {:global, :mining_server}
    • This node is singluar in the network and responsible for distributing work among its children which are the process pool managers from remote nodes
  • Each node has a locally named process pool backed by the GenServer MinerPool, which recieves work messages from the master node MiningServer and translates them appropriately for its children to do work with. All commuincation is async with a callback to a master node’s listening process
  • The leaf in this tree of messages is a Miner node which does the real heavy lifting in computing the SHA256 of a range of numbers to find valid coin-strings

Drawn as a tree, the network looks like so (arrows indicate flow of messages)

                  MiningServer
                      |
                      V
------------------------------------------------------
|(Node1)     | (Node2)    | (Node 3)   ... | (Node n)
|->MinerPool |->MinerPool |->MinerPool     |->MinerPool
  |->Miner     |->Miner     |->Miner         |->Miner
  |->Miner     |->Miner     |->Miner         |->Miner
  ...          ...          ...              ...
  |->Miner     |->Miner     |->Miner         |->Miner
  ---------------------------------------------------
                      |-> MiningServer


PS: If this forum is not the right one for this kind of a topic, please direct me to one that may help in this regard! Really excited to learn about

Most Liked

kylethebaker

kylethebaker

A few inconsequential things that I noticed:

Instead of:

def start_link do
  GenServer.start_link(__MODULE__, [])
end

def start_link(_) do
  GenServer.start_link(__MODULE__, [])
end

You can use default parameters and have the compiler create the two functions for you:

def start_link(_opts \\ [])

I also noticed a few places you are using using the __MODULE__ macro to call functions from the current module, like:

__MODULE__.add_workers(pid, workers)

__MODULE__ gets expanded at compile time to whichever module is currently being compiled, so the above is the same as just calling the function directly since you are already in the current modules scope:

add_workers(pid, workers)

The git repo is missing the Paras.MiningSupervisor supervisor so I’m not able to run it or work out exactly how your tree is set up currently. I don’t have too much experience in this area, but I could picture a supervision tree like such:

<Main Supervisor>
    MiningServer
    <Pools Supervisor>
        <Miner Pool Supervisor>[Node 1]
            MinerPool
            <Worker Supervisor>
                Miners...
        <Miner Pool Supervisor>[Node 2]
            MinerPool
            <Worker Supervisor>
                Miners...
        <Miner Pool Supervisor>[Node N]
            MinerPool
            <Worker Supervisor>
                Miners...

This way each of the individual workers in a pool are supervised together separately from the MinerPool they communicate with. You also have each pool in its own supervision tree that can run on each node, this is for dealing with MinerPool crashes. Then each pool in its entirety is supervised so they can crash independently from other pools. An finally you have the main supervisor managing the MiningServer and all of the pools.

I’m not totally confident this is the “right way”, so take this approach with a grain of salt. Hopefully someone else will chime in to clarify. I think generally when you have a pool of workers and some server module that manages them you want to put the workers under their own supervisor, and then have {ManagerServer, WorkerSupervisor} under their own supervisor.

Last Post!

theyenaman

theyenaman

I had accidentally added mining_supervisor.ex to my gitignore file while excluding the binary. You can try building it now. Sorry for the confusion!

I’ll try re-implementing the supervision tree to match what you said. It makes more sense that each logical unit should have its own boss looking after the underlings. Thanks for taking the time to respond!

Where Next?

Popular in Discussions Top

PragTob
Hey everyone, this has been on my mind for some time and I’d love your input on it! TLDR: I feel like maps are superioer for storing and...
New
scouten
I’m looking for a host for the server part of a small (personal) side project that I’m working on. It’s currently written in Node.js and ...
New
tmbb
This is a post to discuss the new Phoenix LiveView functionality. From Chris’s talk, it appears that they generate all HTML on the serve...
342 18634 126
New
AstonJ
Seen any cool LiveView demos, sample apps or examples? Please post them here! :003:
New
RudManusachi
What configs will make sense to put to runtime.exs? – A bit of how I configure apps: I have generic configs in config/config.exs, dev...
New
saif
Hello everyone, Long time lurker first time poster here. I’ve recently begun working on Elixir full-time again! :raised_hands: It’s been...
New
mmmrrr
Just saw that dhh announced https://hotwire.dev/ Is it just me or is this essentially live view? :smiley: Although I like the “iFrame-e...
New

Other popular topics Top

Qqwy
Update: How to use the Blogs &amp; Podcasts section You can post links to your blog posts or podcasts either in one of the Official Blog...
3271 131117 1222
New
grych
Hi folks, Few months ago I have announced the proof-of-concept of the library to manipulate the browsers DOM objects directly from Elixi...
639 54260 488
New
Qqwy
Original source of discussion: This topic on the Pragmatic Programmers’ Functional Web Development with Elixir, OTP, and Phoenix forum. ...
New
New
nsuchy
Hi. I’ve noticed that Windows Powershell has it’s own IEX command and you cannot access Elixir’s IEX due to the conflict. This isn’t a cr...
New
aadeshere1
I have a another noob question about loop. Since elixir is immutable, while loop is not directly possible. total = 10 while total != 0 ...
New

We're in Beta

About us Mission Statement