plausible / ecto_ch

Ecto ClickHouse adapter
MIT License
72 stars 10 forks source link

Schema generation not connecting to custom url #82

Closed daviaws closed 1 year ago

daviaws commented 1 year ago

I tried to get my schemas as said in the README, but I found this error: image

After editing the code: image

And run: mix deps.compile ecto_ch

I still had to run like this: mix ecto.ch.schema vehicle_summaries_replicated

To get the real result.

ruslandoga commented 1 year ago

Yeah, mix ecto.ch.schema is at PoC stage right now, I'll try and add config lookup from Repo today.

ruslandoga commented 1 year ago

I think it should be fixed in https://github.com/plausible/ecto_ch/pull/83

@daviaws could you please try it out? If it works, I'll cut a release with that fix later today.

You'd need to update deps/0 in your mix.exs

def deps do
  [
     ...
-    {:ecto_ch, "~> 0.1"},
+    {:ecto_ch, github: "plausible/ecto_ch", branch: "repo-config-ecto-ch-schema"}
     ...
  ]
end
daviaws commented 1 year ago

Thank you! I'll check soon!!

daviaws commented 1 year ago

I reviewed and left a comment. But great work. I think after that you can close this one :)