v0.2 multi-statement queries broken? #83

Closed
CnApTaK opened this Issue Oct 15, 2012 · 7 comments

3 participants

@CnApTaK

conn.queryRaw('select 1 test; select 2 test;', function (err, res, nextResult) {
if (err) {
console.log(err);
} else {
console.log(res);
console.log(nextResult);
}
});
res => { meta: [ { name: 'test', size: 10, nullable: false, type: 'number' } ], rows: [ [ 1 ] ] }
nextResult => undefined

@jkint

Strange. I get the following results:

{ meta: [ { name: 'test', size: 10, nullable: false, type: 'number' } ],
  rows: [ [ 1 ] ] }
true
{ meta: [ { name: 'test', size: 10, nullable: false, type: 'number' } ],
  rows: [ [ 2 ] ] }
false

with the following script (which I believe is functionally equivalent to the one you're using):

sql.open( conn_str, function( e, conn ) {

    assert.ifError( e );

    conn.queryRaw( 'select 1 test; select 2 test;', function (err, res, nextResult) {

        if (err) {
           console.log(err);
        } else {
            console.log(res);
            console.log(nextResult);
        }
    });
});

I built this off master, which has 0.2 merged into it, using node 0.6 x86 and a complete rebuild:

git checkout master
node-gyp rebuild
copy build\release\sqlserver.* lib

Please let us know what configuration you're using and, if possible, the commit would be nice too.

Thanks.

@CnApTaK

i am using latest node release - 0.8.12 x64
and when i run npm install node-sqlserver i get some warnings:
npm WARN deprecated node-sqlserver@0.1.0: This package has been superseded by msnodesql.
npm WARN engine node-sqlserver@0.1.0: wanted: {"node":"0.6"} (current: {"node":"v0.8.12","npm":"1.1.63"})

node-sqlserver@0.1.0 install ...\node_modules\node-sqlserver
node-gyp rebuild

this mean i have version 0.1.0 installed? :)
how to install via npm version 0.2?

@CnApTaK

only working with manual install.. zip from https://github.com/WindowsAzure/node-sqlserver/zipball/master.. build.. etc
so problem is with node version?

@jkint

Yes, npm will be updated in the next day or so. Unfortunately for now only the manual installation via github is working properly.

Sorry for the inconvenience.

@CnApTaK

you don't need to be sorry... i'm blind sometimes.. you are doing great tool

@CnApTaK CnApTaK closed this Oct 15, 2012
@jguerin

NPM is already updated. You should now use msnodesql.

Thanks,

Jonathan

@CnApTaK

what about http://www.microsoft.com/en-us/download/details.aspx?id=29995 ? this sfx contain only 32bit version?

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