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

docs: AP-5783 add QuickSight user docs #476

Merged
merged 5 commits into from
Dec 17, 2024

Conversation

tom-webber
Copy link
Contributor

@tom-webber tom-webber commented Dec 11, 2024

@tom-webber tom-webber requested review from a team as code owners December 11, 2024 16:59
style: move external links to bottom of pages for easier markdown parsing and link reuse
style: renamed markdown files to reflect file order/hierarchy
Comment on lines 22 to 28
You'll be asked if you want to 'Import to [SPICE] for quicker analytics'.
This imports the data into memory, rather than querying the data where it lives in S3. Whether to use SPICE or directly query the data is a judgement call, and will likely come down to:

- how complex the analysis is
- the size, format and structure of the underlying data
- how likely users are to drill down into the dashboards
- user tolerance for load times while making changes to dashboards

Choose a reason for hiding this comment

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

Is it possible to flip this approach? For example, giving users a recommendation for the factors you've described. 'You should not use SPICE if...'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I like this idea. My suggestion would basically boil down to 'SPICE costs money, don't use it if you don't need it'. Due to lack of familiarity with the impacts it makes on user experience, I don't feel knowledgable enough for that approach, and would prefer to leave it in the users' hands.

Copy link
Contributor

@julialawrence julialawrence left a comment

Choose a reason for hiding this comment

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

LGTM

@tom-webber tom-webber merged commit 9429e23 into main Dec 17, 2024
1 check passed
@tom-webber tom-webber deleted the docs/AP-5783-add-quicksight-user-docs branch December 17, 2024 11:53
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.

3 participants