Distinguish queries made by phoenix_ecto about phoenix_ecto HOT 9 CLOSED

Oh, I understand now. IIRC, we started adding some option for migration queries, something like :migration_source? Please inspect all telemetry metadata and see if there is something you can use.

If not, I think you can pass telemetry_options: %{dev_logger: false} to the Plug too.

Finally, you should consider sending the improvements in ecto_dev_logger upstream. I know @v0idpwn was excited about the package. At least the coloring changes should be a no-brainer. We would love to have coloring for stacktrace, for the db timing, etc. :)

from phoenix_ecto.

I think you can pass log: false to the check status plug? If so, can you please:

1. Document the log: false option
2. Change Phoenix generators to pass log: false by default?

from phoenix_ecto.

Sorry, I didn't get the idea with log: false.
log: false is a requirement to use my custom logger in order not to duplicate logs.
According to this code
https://github.com/elixir-ecto/ecto_sql/blob/c73ba2b1edf9d049804c2103d7dd4c9d28e6c59d/lib/ecto/adapters/sql.ex#L1107-L1120
telemetry event is published even if log: false

from phoenix_ecto.

Please inspect all telemetry metadata and see if there is something you can use.

Nope, here is one small example of metadata

%{
  options: [],
  params: [],
  query: "commit",
  repo: MyApp.Repo,
  result: {:ok,
   %Postgrex.Result{
     columns: nil,
     command: :commit,
     connection_id: 68112,
     messages: [],
     num_rows: nil,
     rows: nil
   }},
  source: nil,
  stacktrace: nil,
  type: :ecto_sql_query
}

Passing additional options to the Plug also does not help.

We would love to have coloring for stacktrace, for the db timing, etc. :)

Good to know, I'll open a discussion later :)

from phoenix_ecto.

Passing additional options to the Plug also does not help.

We should support that, so PRs here are welcome too!

from phoenix_ecto.

@fuelen you should now be able to see schema_migration: true in the telemetry options using ecto_sql master

from phoenix_ecto.

@greg-rychlewski I tried it and schema_migration: true is present only for these log lines:

CREATE TABLE IF NOT EXISTS "schema_migrations" ("version" bigint, "inserted_at" timestamp(0), PRIMARY KEY ("version"))

SELECT s0."version"::bigint FROM "schema_migrations" AS s0

so, the option is absent for begin, commit and LOCK TABLE ...

from phoenix_ecto.

Ah, yes. Those are in different files and we need to set them too.

from phoenix_ecto.

@fuelen Thanks for catching that. Now it should be fixed in ecto_sql master.

from phoenix_ecto.