Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Merged
windoze merged 1 commit into
mainfeathr-ai/feathr:mainfrom
blair/updates-on-docfeathr-ai/feathr:blair/updates-on-docCopy head branch name to clipboard
Jan 10, 2023
Merged

Updates on docs to remove out-of-date contents#950
windoze merged 1 commit into
mainfeathr-ai/feathr:mainfrom
blair/updates-on-docfeathr-ai/feathr:blair/updates-on-docCopy head branch name to clipboard

Conversation

@blrchen
Copy link
Copy Markdown
Collaborator

@blrchen blrchen commented Jan 10, 2023

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 windoze merged commit e1895bb into main Jan 10, 2023
@blrchen blrchen deleted the blair/updates-on-doc branch January 10, 2023 06:55
@jainr
Copy link
Copy Markdown
Collaborator

jainr commented Jan 27, 2023

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

@blrchen
Copy link
Copy Markdown
Collaborator Author

blrchen commented Jan 27, 2023

@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?

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.