Skip to content
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.

Sign up for GitHub

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

Jump to bottom

tidb: Fix panic in driverStmt.Query #63

Merged
merged 5 commits into from
Sep 8, 2015
Merged

tidb: Fix panic in driverStmt.Query #63

merged 5 commits into from
Sep 8, 2015

Commits on Sep 8, 2015

  1. tidb: Fix panic in driverStmt.Query 

    The prepared statement maybe not query, so the resultset maybe nil.
    @shenli
    shenli committed Sep 8, 2015
    Configuration menu
    Copy the full SHA
    67d64d6 View commit details
    Browse the repository at this point in the history

  2. tidb: Return empty rows for exec statement 

    Address comment
    @shenli
    shenli committed Sep 8, 2015
    2 Configuration menu
    Copy the full SHA
    a71acae View commit details
    Browse the repository at this point in the history

  3. tidb: Add test case for empty rows 

    Address comment
    @shenli
    shenli committed Sep 8, 2015
    1 Configuration menu
    Copy the full SHA
    35fd2d1 View commit details
    Browse the repository at this point in the history

  4. tidb: Add test case for empty rows 

    1. Close()
2. Call Next() two times
    @shenli
    shenli committed Sep 8, 2015
    1 Configuration menu
    Copy the full SHA
    1329e77 View commit details
    Browse the repository at this point in the history

  5. tidb: Address comment

    @shenli
    shenli committed Sep 8, 2015
    Configuration menu
    Copy the full SHA
    663e958 View commit details
    Browse the repository at this point in the history