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

Doc fixes for query from deep storage and MSQ #15313

Merged
merged 4 commits into from
Nov 3, 2023

Conversation

cryptoe
Copy link
Contributor

@cryptoe cryptoe commented Nov 2, 2023

Minor updates to the documentation.

  • Added pre requisites.
  • Removed a known issue in MSQ since its no longer valid.

@cryptoe cryptoe requested a review from 317brian November 2, 2023 17:39
@cryptoe cryptoe added this to the 28.0 milestone Nov 2, 2023
Copy link
Contributor

@317brian 317brian left a comment

Choose a reason for hiding this comment

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

Minor nits but LGTM besides that

docs/querying/query-from-deep-storage.md Outdated Show resolved Hide resolved
cryptoe and others added 3 commits November 3, 2023 09:22
Co-authored-by: 317brian <53799971+317brian@users.noreply.github.com>
Co-authored-by: 317brian <53799971+317brian@users.noreply.github.com>
Co-authored-by: 317brian <53799971+317brian@users.noreply.github.com>
@cryptoe cryptoe merged commit 5036af6 into apache:master Nov 3, 2023
11 checks passed
cryptoe added a commit to cryptoe/druid that referenced this pull request Nov 3, 2023
Minor updates to the documentation.

    Added prerequisites.
    Removed a known issue in MSQ since its no longer valid.

---------

Co-authored-by: 317brian <53799971+317brian@users.noreply.github.com>
(cherry picked from commit 5036af6)
abhishekagarwal87 pushed a commit that referenced this pull request Nov 3, 2023
Minor updates to the documentation.

    Added prerequisites.
    Removed a known issue in MSQ since its no longer valid.

---------

Co-authored-by: 317brian <53799971+317brian@users.noreply.github.com>
(cherry picked from commit 5036af6)
CaseyPan pushed a commit to CaseyPan/druid that referenced this pull request Nov 17, 2023
Minor updates to the documentation.

    Added prerequisites.
    Removed a known issue in MSQ since its no longer valid.

---------

Co-authored-by: 317brian <53799971+317brian@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants