Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Migrate plugin to use sqlds (requires a golang SQL driver) and the aws-sdk SQL packages. #130

Closed
Tracked by #124
andresmgot opened this issue Jan 27, 2022 · 2 comments
Closed
Tracked by #124

Migrate plugin to use sqlds (requires a golang SQL driver) and the aws-sdk SQL packages. #130

andresmgot opened this issue Jan 27, 2022 · 2 comments
Assignees
@andresmgot
Labels
datasource/Timestream

Comments

@andresmgot
Copy link
Contributor

No description provided.

@andresmgot andresmgot mentioned this issue Jan 27, 2022
Closed
2 tasks
@andresmgot andresmgot self-assigned this Jan 27, 2022
@andresmgot andresmgot mentioned this issue Jan 27, 2022
Merged
@andresmgot
Copy link
Contributor Author

After some investigation, we have decided to not move forward with the integration of sqlds and the SQL package of the aws-sdk because the plugin currently support:

  • Non SQL fields like time series, arrays and rows.
  • Several time series per query
  • Custom labeling
  • Paginated responses

The libraries are not adapted to handle that (and they probably shouldn't, for some of the items, to be generic). More information about this in this document.

@taleena
Copy link

taleena commented Apr 1, 2022

Effort for this was captured in #131 (and was originally an estimate prior to doing the work).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andresmgot andresmgot

Labels
datasource/Timestream
Projects
AWS Datasources
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@andresmgot @taleena @vickyyyyyyy