1212579 - Integrate raptor-query into raptor-cli

Status: RESOLVED FIXED

Description

[:Eli Perelman]

Allow making queries to an InfluxDB datasource via the CLI through use of the raptor-query module.

Comment 1

[:Eli Perelman]

Created attachment 8696976 [details] [review]
PR: https://github.com/mozilla-raptor/query/pull/1

Comment 2

[Robert Wood [:rwood]]

Comment on attachment 8696976 [details] [review]
PR: https://github.com/mozilla-raptor/query/pull/1

R+ on condition that your .raptorrc patch lands in the CLI, so that query works with .raptorrc settings:

https://gist.githubusercontent.com/eliperelman/e3c12a83a92e820d8b8a/raw/ff69e4395441f738ded7c25e78bb55ba5017989a/file.patch

Comment 3

[:Eli Perelman]

RC config patch landed.

Comment 4

[:Eli Perelman]

First PR in master: https://github.com/mozilla-raptor/query/commit/23fcd813b0c52be350f7491bd316f7d6782a10dc

Comment 5

[:Eli Perelman]

Created attachment 8699062 [details] [review]
PR: https://github.com/mozilla-raptor/query/pull/2

Follow-up patch to make required parameter validation better.

Comment 6

[Robert Wood [:rwood]]

Comment on attachment 8699062 [details] [review]
PR: https://github.com/mozilla-raptor/query/pull/2

LGTM and works great

Comment 7

[:Eli Perelman]

Second PR in master: https://github.com/mozilla-raptor/query/commit/71588d09bec05f4c83f661934dc84b637534d056

Leaving open until I do the npm publishing.

Comment 8

[:Eli Perelman]

In npm: raptor-query@4.0.0