-
Notifications
You must be signed in to change notification settings - Fork 94
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
ParallelStream is not available #56
Comments
Probably you want to load parallel stream as part of your applications, however that should be done by this library in the current version - which version are you using? |
From mix.lock
Yeah, I thought about that. I experimented loading parallel stream in my own application as an application dependency. The application wouldn't start since parallel stream isn't a direct dependency. |
What does your application definition in mix.exs look like, does it load csv? CSV itself loads parallel_stream as an application dependency, so it should be loaded as part of that. |
Here is an except of my mix file: def application do
[mod: {Aprit, []},
applications: [:phoenix, :phoenix_pubsub, :phoenix_html, :cowboy, :logger, :gettext,
:phoenix_ecto, :postgrex, :guardian, :csv, :edeliver]]
end
defp deps do
[{:phoenix, "~> 1.2.1"},
{:phoenix_pubsub, "~> 1.0"},
{:phoenix_ecto, "~> 3.0"},
{:postgrex, ">= 0.0.0"},
{:phoenix_html, "~> 2.6"},
{:phoenix_live_reload, "~> 1.0", only: :dev},
{:guardian, "~> 0.12.0"},
{:gettext, "~> 0.11"},
{:poison, "~> 1.0"},
{:csv, "~> 1.4.2"},
{:cowboy, "~> 1.0"},
{:edeliver, "1.3.0"},
{:distillery, ">= 0.8.0", warn_missing: false}]
end Oh I see that it does require parallel stream as an application dependency. Hmmm, weird. I'm probably doing something wrong. If you don't mind, I'll go ahead close the issue for now and I'll reopen it if I find anything else. |
@luk3thomas Did you find a solution? I have this same issue and have already tried adding |
Might this be related to this issue edeliver/edeliver#157 ? |
@beatrichartz Thanks, but I was able to build my releases. |
I think #51 should fix the problem I am seeing. I suppose the fix will be included in an upcoming release sometime soon? |
@luk3thomas I released this in 1.4.4 today, sorry for the wait |
Hi!
I recently created release with edeliver and I'm seeing this?
Naive question, but would requiring
ParallelStream
in theDecoder
module fix the runtime error I am seeing?The text was updated successfully, but these errors were encountered: