elixir-ls icon indicating copy to clipboard operation
elixir-ls copied to clipboard
verified publisher icon elixir-lsp

Start the beam instance with the ability to cluster

Open axelson opened this issue 6 years ago • 2 comments

This probably isn't good to merge as-is. I'm thinking that auto-clustering should be opt-in based on a configuration settings file. Does anyone have thoughts about this? Another option is that I could just keep this as a patch locally for my own debugging.

EDIT: Updated to not auto-cluster but still start up distribution

axelson avatar Aug 25 '19 22:08 axelson

This has already been proved useful for tracking a memory leak. I think this is ok to be merged once it's conflicts are resolved.

victorolinasc avatar Dec 30 '19 20:12 victorolinasc

I agree that we should merge it but I would like to make the following changes first:

  • [ ] Make this opt-in via a configuration setting
  • [ ] Bug where if you start up ElixirLS with this patch and haven't started epmd ElixirLS will fail to startup

axelson avatar Dec 31 '19 18:12 axelson

I don't think this PR is needed anymore. Node (s)name can be passed via env and then ElixirLS starts with ability to cluster

lukaszsamson avatar Sep 25 '23 19:09 lukaszsamson