Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: Weaviate destination connector #1963

Merged
merged 47 commits into from
Dec 1, 2023
Merged

feat: Weaviate destination connector #1963

merged 47 commits into from
Dec 1, 2023

Conversation

rvztz
Copy link
Contributor

@rvztz rvztz commented Oct 31, 2023 •
edited
Loading

Closes #1781.

  • Adds a Weaviate destination connector
  • The connector receives a host for the weaviate instance and a weaviate class name.
  • Defines a weaviate schema for json elements.
  • Defines the pre-processing to conform unstructured's schema to the proposed weaviate schema.

@rvztz rvztz temporarily deployed to ci October 31, 2023 21:39 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci October 31, 2023 21:39 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci October 31, 2023 21:39 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 10:56 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 10:58 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 10:58 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 10:58 — with GitHub Actions Inactive
@rvztz rvztz marked this pull request as ready for review November 1, 2023 20:55
@rvztz rvztz requested a review from rbiseck3 November 1, 2023 20:55
@ryannikolaidis
Copy link
Contributor

ryannikolaidis commented Nov 1, 2023 •
edited
Loading

please add the pr description and make the pr title a conventional commit name (since this will be used for the commit by merge queue)

@rvztz rvztz temporarily deployed to ci November 1, 2023 21:24 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 21:29 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 21:29 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 21:29 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 21:29 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 21:29 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 21:29 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 1, 2023 21:29 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 2, 2023 01:47 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 2, 2023 01:49 — with GitHub Actions Inactive
@rvztz rvztz temporarily deployed to ci November 2, 2023 01:49 — with GitHub Actions Inactive
rbiseck3
rbiseck3 approved these changes Nov 29, 2023
View reviewed changes
Copy link
Contributor

@rbiseck3 rbiseck3 left a comment

Choose a reason for hiding this comment

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

One small comment, other LGTM.

@ryannikolaidis ryannikolaidis added this pull request to the merge queue Dec 1, 2023
Merged via the queue into main with commit ce905dd Dec 1, 2023
50 checks passed
@ryannikolaidis ryannikolaidis deleted the rvztz/weaviate-destination-connector branch December 1, 2023 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ahmetmeleq ahmetmeleq ahmetmeleq left review comments

@potter-potter potter-potter potter-potter left review comments

@rbiseck3 rbiseck3 rbiseck3 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat: create Weaviate Destination Connector
6 participants
@rvztz @ryannikolaidis @rbiseck3 @ahmetmeleq @mgssf @potter-potter