Skip to content

Conversation

@Yuqing-cat
Copy link
Collaborator

Signed-off-by: Yuqing Wei [email protected]

Description

Resolves #882

How was this PR tested?

Does this PR introduce any user-facing changes?

  • No. You can skip the rest of this section.
  • Yes. Make sure to clarify your proposed changes.

@Yuqing-cat Yuqing-cat added the safe to test Tag to execute build pipeline for a PR from forked repo label Nov 28, 2022
@Yuqing-cat Yuqing-cat changed the title add sql credential pass through doc and test case add sql credential pass through doc Nov 29, 2022
Signed-off-by: Yuqing Wei <[email protected]>
@Yuqing-cat Yuqing-cat requested a review from blrchen December 1, 2022 05:33
Signed-off-by: Yuqing Wei <[email protected]>
@xiaoyongzhu xiaoyongzhu merged commit 9077f00 into feathr-ai:main Dec 2, 2022
jaymo001 pushed a commit that referenced this pull request Dec 7, 2022
* add sql credential pass through doc

Signed-off-by: Yuqing Wei <[email protected]>

* fix comments

Signed-off-by: Yuqing Wei <[email protected]>

* fix comments

Signed-off-by: Yuqing Wei <[email protected]>

Signed-off-by: Yuqing Wei <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test Tag to execute build pipeline for a PR from forked repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOC] Add instruction of SQL Credential Pass through with Access Token

3 participants