Skip to content

Conversation

mattn
Copy link
Owner

@mattn mattn commented Nov 4, 2016

Closing #346

On Conn

  • BeginContext(ctx) (tx,err) found in ConnBeginContext
  • PrepareContext(ctx, query) (stmt,err) found in ConnPrepareContext
  • ExecContext(ctx, query, []namedvalue) (res, err) found in ExecerContext
  • Ping(ctx) err found in Pinger
  • QueryContext(ctx, query, []namedvalue) (rows, err) found in QueryerContext

On "Stmt"

  • ExecContext(ctx, []namedvalue) (res, err) found in StmtExecContext
  • QueryContext(ctx, []namedvalue) (rows, err) found in StmtQueryContext

On "Rows"

  • HasNextResultSet() bool found in RowsNextResultSet
  • NextResultSet() error found in RowsNextResultSet
  • ColumnTypeDatabaseTypeName(index int) (name string)
  • ColumnTypeLength(index int) (length int64, ok bool)
  • ColumnTypeNullable(index int) (nullable, ok bool)
  • ColumnTypePrecisionScale(index int) (precision, scale int64, ok bool)
  • ColumnTypeScanType(index int) reflect.Type

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.9%) to 59.806% when pulling 29b191f on ping into 86681de on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.9%) to 59.806% when pulling 025b917 on ping into 86681de on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.7%) to 60.0% when pulling 57eaf4c on ping into 86681de on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.1%) to 62.773% when pulling 57eaf4c on ping into 86681de on master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+1.1%) to 62.773% when pulling 57eaf4c on ping into 86681de on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.1%) to 62.773% when pulling 57eaf4c on ping into 86681de on master.

@coveralls
Copy link

coveralls commented Nov 4, 2016

Coverage Status

Coverage increased (+1.1%) to 62.773% when pulling 57eaf4c on ping into 86681de on master.

@coveralls
Copy link

coveralls commented Nov 4, 2016

Coverage Status

Coverage increased (+1.1%) to 62.773% when pulling 57eaf4c on ping into 86681de on master.

@mattn mattn changed the title new go18 feature new go1.8 feature Nov 4, 2016
@mattn
Copy link
Owner Author

mattn commented Nov 4, 2016

I decide NOT to support NextResultSet/HasNextResultSet because the go's multiple result is not same as sqlite's one.

@mattn mattn merged commit f4e49e6 into master Nov 4, 2016
@azavorotnii
Copy link

@mattn is there some workaround for NextResultSet usage? Currently, if I have multiple selects in same Query() it returns only last select rows.

@mattn mattn deleted the ping branch November 28, 2017 07:28
@mattn
Copy link
Owner Author

mattn commented Nov 28, 2017

When I looked at database / sql before, it seemed impossible to implement multiple-resultset for sqlite3 because the behavior of multiple-resultset of database/sql is different from the one expected. But now I looked it, I seems possible to implement. (Seems HasNextResultSet is gone, but not problem)

@mattn
Copy link
Owner Author

mattn commented Nov 28, 2017

#497

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants