-
Notifications
You must be signed in to change notification settings - Fork 8
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.
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
Add client support for NLPRP tabular schema #180
base: master
Are you sure you want to change the base?
Conversation
This reverts commit fb5d508.
This change implements the client aspects of the There's an end-to-end test in the test suite that creates an SQLite database (requiring a few minor tweaks) and sets up a client-server pair. The client asks the server for its NLP definitions, then sets up a couple of database tables based on that. To be safe, remote table names aren't trusted, but if you map a remote table name to an empty definition, it'll use the remote |
We'd designed the specification, but not implemented it properly client-side. Now this matters!