-
Notifications
You must be signed in to change notification settings - Fork 743
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.
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
Adds data source properties to git connectors #1280
Conversation
… update (#1344) This pull request includes updated ingest test fixtures. Please review and merge if appropriate. Co-authored-by: rvztz <[email protected]>
… update (#1358) This pull request includes updated ingest test fixtures. Please review and merge if appropriate. Co-authored-by: rvztz <[email protected]>
…IO/unstructured into data-source-properties-git
test_unstructured_ingest/expected-structured-output/github/LICENSE.txt.json
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see note on record_locator
… update (#1491) This pull request includes updated ingest test fixtures. Please review and merge if appropriate. Co-authored-by: rvztz <[email protected]>
test_unstructured_ingest/expected-structured-output/github/LICENSE.txt.json
Outdated
Show resolved
Hide resolved
… update (#1554) This pull request includes updated ingest test fixtures. Please review and merge if appropriate. Co-authored-by: rvztz <[email protected]>
…IO/unstructured into data-source-properties-git
@@ -1,28 +0,0 @@ | |||
[ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks like a bug, this shouldn't be removed. try re-running fixtures.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Adds data source properties to git connectors:
These properties are instantiated when supported by the connector.
Separates the logic between fetching the file from source and
get_file
. Retrieves file metadata when any of the properties are called.Adds logic to check if file exists in the remote source. For connectors that don't directly support it, adds exception handling to check any issues while retrieving the file.