- allow "exec" commmand to pass dictionary-params to webworker (with testcase)

[kaizhu256]

• add test passing dictionary-params to webworker

i generally use sql.js via webworkers, b/c i need the browser-ui to be responsive while ingesting/manipulating largish (200mb) csv's and queries that could take 30+ seconds.

the biggest missing feature for me using webworkers is passing parameters. i've been successfully using the patch in this pull-request in production for the past 6 months.

[lovasoa]

This looks good to me. But since it's a new feature, we need to document it.

[kaizhu256]

added documentation for multi-line exec-statements with params in commit # 2: 7bdff4d.

also verified documented example-usage runs correctly in browser:

[kaizhu256]

log of commits-notes:

- allow "exec" commmand to pass dictionary-params to webworke
- add test passing dictionary-params to webworker
- add documentation
- resolve conversation - Statement.BindParams
- resolve conversation - replace "several queries" with "several statements"
- try to limit documentation to 80-column width

1 remaining unresolved conversation: example use-case @ line 178 in src/api.js

[lovasoa]

I don't see any unresolved conversation. Is this ready for merging?

[kaizhu256]

yes, its ready for merge on my side! : )

i have another worker patch in the works (to ingest large csv's while keeping ui-responsive) that depends on this.

[kaizhu256]

re-opening this convesation, since it was clobbered by last commit. is example-code ok the way it is, or still too verbose? verified example works.

[lovasoa]

Ok, it's merged. Thank you @kaizhu256. What is your other patch ? Maybe we can discuss it first ? I was thinking about deprecating our buggy and largely untested worker code in favor of more modern and feature-complete alternatives.