Skip to content

Add user mapping configuration instruction#11741

Merged
hashhar merged 1 commit intotrinodb:masterfrom
Jessie212:jt/user-mapping
Apr 25, 2022
Merged

Add user mapping configuration instruction#11741
hashhar merged 1 commit intotrinodb:masterfrom
Jessie212:jt/user-mapping

Conversation

@Jessie212
Copy link
Contributor

Description

Is this change a fix, improvement, new feature, refactoring, or other?

Improvement.

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

Change to documentation only.

How would you describe this change to a non-technical end user or system administrator?

Add configuration instruction.

Related issues, pull requests, and links

Documentation

( ) No documentation is needed.
(x) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required.
( ) Release notes entries required with the following suggested text:

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

@cla-bot cla-bot bot added the cla-signed label Mar 31, 2022
@github-actions github-actions bot added the docs label Mar 31, 2022
@findepi findepi changed the title Add configuration instruction Add user mapping configuration instruction Apr 1, 2022
@Jessie212 Jessie212 requested review from Ordinant and hashhar April 4, 2022 14:24
Copy link
Member

@Ordinant Ordinant left a comment

Choose a reason for hiding this comment

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

Comment left with big mea culpa.

@mosabua
Copy link
Member

mosabua commented Apr 21, 2022

@Jessie212 can you verify and confirm that the docs build locally .. CI seems to have some weird (hopefully unrelated failure)

@Jessie212
Copy link
Contributor Author

@mosabua I'm having no issues building this locally.

Copy link
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.

Looks good (an optional commment), will merge once CI is done.

@hashhar hashhar merged commit ab7bc3f into trinodb:master Apr 25, 2022
@github-actions github-actions bot added this to the 379 milestone Apr 25, 2022
@Jessie212 Jessie212 deleted the jt/user-mapping branch May 9, 2022 13:49
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.

4 participants