alibaba / nquery

Generic SQL engine for Web and Big-data.
381 stars 117 forks source link

Breaking changes from 0.3.3->0.3.4 #22

Open cgvedant opened 5 years ago

cgvedant commented 5 years ago

This package works as expected when you install it through npm(v0.3.3) but when I cloned the repo and tried to run demo.js I get the following error when it tries to evaluate $c := ...

Error: no registered db info found...
    at load (/Users/root/nquery/demo/loader.js:44:8)

The problem seems to be with the AST creation. In the newer version the AST doesn't show the tables a and b used for evaluation as 'var' type. Rolling back to the commit 395487a833d0f704a16f4a898dd9a554c94a6e76 fixed the issue.