alibaba / alibaba/nquery

Breaking changes from 0.3.3->0.3.4

Open
#22 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
385
Forks
117
PR merge metrics
No merged PRs in 30d

Description

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.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by running demo.js and inspect the failure in demo/loader.js at line 44 when evaluating `$c := ...`. Compare the AST for tables `a` and `b` with the behavior at commit 395487a833d0f704a16f4a898dd9a554c94a6e76. Done means the tables are represented as `var` for evaluation and demo.js no longer reports missing database info.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
databases
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.