Is this `seeds.exs` idiomatic Elixir?

[riebeekn]

Honestly re-reading what you’re going for I’d probably just stick with the normal seeds file. If you want to control what enviros they get applied to here is another approach (and all around a good article on seeding): https://medium.com/geekculture/phoenix-understanding-how-to-seed-data-c2be863ddd1d

[gregvaughn]

@coen.bakker

Also, I didn’t do any error handling. Is that an issue?

“It depends”

For a script like this, I’d at least want it to be loud, like raising an exception if something goes wrong. I likely wouldn’t do serious error handling to give the user some nice message because the audience is software developers.

[coen.bakker]

@coen.bakker

Enum.chunk_every(book_ids, batch_size)
|> Task.async_stream(
  fn batch ->
    Enum.each(
      batch,
      &Repo.insert_all(Page, build_pages_for_book.(&1))
    )
  end,
  max_concurrency: max_concurrency
)

To avoid loose ends: This code snippet doesn’t work as intended because the async stream never actually runs.

Fixed version: xlp-book-club-API/example/priv/repo/seeds.exs at main · elixirland/xlp-book-club-API · GitHub