Skip to content

Document that adjusting the auto-discovered schema is required#18784

Merged
mosabua merged 1 commit intotrinodb:masterfrom
Jessie212:jt/auto-discovered-schema
Aug 28, 2023
Merged

Document that adjusting the auto-discovered schema is required#18784
mosabua merged 1 commit intotrinodb:masterfrom
Jessie212:jt/auto-discovered-schema

Conversation

@Jessie212
Copy link
Copy Markdown
Contributor

Description

Additional context and related issues

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Aug 23, 2023
@Jessie212 Jessie212 requested review from hashhar and mosabua August 23, 2023 20:38
@github-actions github-actions bot added the docs label Aug 23, 2023
Copy link
Copy Markdown
Member

@hashhar hashhar left a comment

Choose a reason for hiding this comment

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

technically correct, I'll let Manfred do the editorial duties

Comment on lines 214 to 215
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: what do you think about: "by executing in the MongoDB Shell"?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I like that.

@Jessie212 Jessie212 force-pushed the jt/auto-discovered-schema branch from a7b057e to 93a79dc Compare August 24, 2023 15:52
@mosabua mosabua merged commit d606efa into trinodb:master Aug 28, 2023
@github-actions github-actions bot added this to the 426 milestone Aug 28, 2023
@Jessie212 Jessie212 deleted the jt/auto-discovered-schema branch August 28, 2023 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants