Skip to content

Commit

Permalink
remove debug logger
Browse files Browse the repository at this point in the history
  • Loading branch information
nakagami committed Apr 3, 2021
1 parent 93ec3d9 commit 4a3d687
Showing 1 changed file with 2 additions and 25 deletions.
27 changes: 2 additions & 25 deletions lib/firebirdex/protocol.ex
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@ defmodule Firebirdex.Protocol do
@moduledoc false
use DBConnection

require Logger

alias Firebirdex.{Query, Result}

defstruct [
Expand All @@ -12,26 +10,21 @@ defmodule Firebirdex.Protocol do

@impl true
def connect(opts) do
Logger.debug "connect() #{inspect(opts)}"
hostname = to_charlist(opts[:hostname])
username = to_charlist(opts[:username])
password = to_charlist(opts[:password])
database = to_charlist(opts[:database])
case :efirebirdsql_protocol.connect(hostname, username, password, database, opts) do
{:ok, conn} ->
Logger.debug "connect() :ok"
{:ok, %__MODULE__{conn: conn}}
{:error, number, reason, _conn} ->
Logger.debug "connect() :error #{reason}"
{:error, %Firebirdex.Error{number: number, reason: reason}}
end
end

@impl true
def disconnect(_reason, state) do
Logger.debug "disconnect()"
:efirebirdsql_protocol.close(state.conn)
Logger.debug "disconnect() :ok"
:ok
end

Expand All @@ -53,7 +46,6 @@ defmodule Firebirdex.Protocol do

@impl true
def handle_prepare(%Query{} = query, _opts, state) do
Logger.debug "handle_prepare() #{query}"
{:ok, stmt} = :efirebirdsql_protocol.allocate_statement(state.conn)
case :efirebirdsql_protocol.prepare_statement(to_string(query), state.conn, stmt) do
{:ok, stmt} ->
Expand Down Expand Up @@ -94,36 +86,30 @@ defmodule Firebirdex.Protocol do
@impl true
def handle_execute(%Query{} = query, params, _opts, state) do
params = Enum.map(params, &convert_param(&1))
Logger.debug "handle_execute() #{inspect(params)}"
case :efirebirdsql_protocol.execute(state.conn, query.stmt, params) do
{:ok, stmt} ->
{:ok, rows, stmt} = :efirebirdsql_protocol.fetchall(state.conn, stmt)
columns = Enum.map(:efirebirdsql_protocol.columns(stmt), &(column_name(&1)))
{:ok, num_rows} = :efirebirdsql_protocol.rowcount(state.conn, stmt)
Logger.debug "handle_execute() :ok"
{:ok, %Query{query | stmt: stmt}, %Result{columns: columns, num_rows: num_rows, rows: rows}, %__MODULE__{state | conn: state.conn}}
{:error, number, reason} ->
Logger.debug "handle_execute() :error #{reason}"
{:error, %Firebirdex.Error{number: number, reason: reason, statement: query.statement}, %__MODULE__{state | conn: state.conn}}
end
end

@impl true
def handle_close(_query, _opts, %{conn: conn}) do
Logger.debug "handle_close()"
{:ok, conn} = :efirebirdsql_protocol.close(conn)
{:ok, %__MODULE__{conn: conn}}
end

@impl true
def handle_declare(query, _params, _opt, state) do
Logger.debug "handle_declare()"
{:ok, query, query.stmt, state}
end

@impl true
def handle_begin(_opts, %{conn: conn}) do
Logger.debug "handle_begin()"
case :efirebirdsql_protocol.begin_transaction(false, conn) do
{:ok, conn} ->
{:ok, %Result{}, %__MODULE__{conn: conn}}
Expand All @@ -134,46 +120,37 @@ defmodule Firebirdex.Protocol do

@impl true
def handle_commit(_opts, %{conn: conn}) do
Logger.debug "handle_commit()"
case :efirebirdsql_protocol.commit(conn) do
:ok ->
Logger.debug "handle_commit() :ok"
{:ok, %Result{}, %__MODULE__{conn: conn}}
{:error, _errno, reason} ->
Logger.debug "handle_commit() :error #{reason}"
{:error, _errno, _reason} ->
{:error, %__MODULE__{conn: conn}}
end
end

@impl true
def handle_rollback(_opts, %{conn: conn}) do
Logger.debug "handle_rollback()"
case :efirebirdsql_protocol.rollback(conn) do
:ok ->
Logger.debug "handle_rollback() :ok"
{:ok, %Result{}, %__MODULE__{conn: conn}}
{:error, _errno, reason} ->
Logger.debug "handle_rollback() :error #{reason}"
{:error, _errno, _reason} ->
{:error, %__MODULE__{conn: conn}}
end
end

@impl true
def handle_status(_opts, s) do
Logger.debug "handle_status()"
# TODO: transaction status treatment
{:transaction, s}
end

@impl true
def handle_fetch(_query, %Result{} = result, _opts, s) do
Logger.debug "handle_fetch()"
{:halt, result, s}
end

@impl true
def handle_deallocate(query, _cursor, _opts, state) do
Logger.debug "handle_deallocate()"
case :efirebirdsql_protocol.free_statement(state.conn, query.stmt, :drop) do
{:ok, _stmt} ->
{:ok, %Result{}, %__MODULE__{conn: state.conn}}
Expand Down

0 comments on commit 4a3d687

Please sign in to comment.