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.

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 ADR for data exploration options #950

Merged
merged 2 commits into from
Oct 27, 2020
Merged

Add ADR for data exploration options #950

merged 2 commits into from
Oct 27, 2020

Conversation

robhudson
Copy link
Member

No description provided.

2. We could provide the SQL to query the current set of data being displayed to
the user which can then be copy/pasted into Redash or BQ console as a
starting point for further exploration
3. We could link directly to Redash with the query pre-populated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Option 3 is pretty much a non-starter... given the BI tool evaluation that is currently happening, we don't know what role Redash will serve going forward, investing in a tighter integration with Redash would be unwise right now.


Extra options we can provide:

1. We could provide other alternate data file formats (e.g. CSV)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could make sense, but IMO only if we identify a concrete workflow that we're trying to serve, which would require some user research / interviews to find out. Are there specific product managers who would like to load the data into Excel to do further crunching? If yes, then adding CSV export sounds great! But we probably shouldn't build it unless we know there's a demand for the format(s) we're adding.

Extra options we can provide:

1. We could provide other alternate data file formats (e.g. CSV)
2. We could provide the SQL to query the current set of data being displayed to
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to me the most useful initial deliverable.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also discovered this today: #634

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants