FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Updates on docs to remove out-of-date contents by blrchen · Pull Request #950 · feathr-ai/feathr · GitHub

Updates on docs to remove out-of-date contents - #950

Merged
windoze merged 1 commit into
mainfrom
blair/updates-on-doc
Jan 10, 2023
Merged

Updates on docs to remove out-of-date contents#950
windoze merged 1 commit into
mainfrom
blair/updates-on-doc

Conversation

blrchen commented Jan 10, 2023

Copy link
Copy Markdown
Collaborator

Description

Updates on docs to remove out of date contents

  • Remove the content for using Elastic search as online store
  • Remove the guidence not use sql registry for Prod

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.

windoze merged commit e1895bb into main Jan 10, 2023
blrchen deleted the blair/updates-on-doc branch January 10, 2023 06:55

jainr commented Jan 27, 2023

Copy link
Copy Markdown
Collaborator

@blrchen - Could you please comment on why we removed the cosmosDBsink section from the docs?

blrchen commented Jan 27, 2023

Copy link
Copy Markdown
Collaborator Author

@blrchen - Could you please comment on why we removed the cosmosDBsink section from the docs?

Hello @jainr , thanks for raising this up. The CosmosDb section doc removal is by mistaken. I only intented to remove the Elastic Search section. Could you please help adding this back in your PRs?

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL