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

Fix for testing #84

Merged
merged 10 commits into from
Nov 1, 2024
Merged

Fix for testing #84

merged 10 commits into from
Nov 1, 2024

Conversation

nitrosx
Copy link
Contributor

@nitrosx nitrosx commented Oct 30, 2024

This PR contains the fixes implemented up until now to run the ingestion successfully.
Here is a rough list:

  • add option to use full file name in file list
  • generate ingestor run id automatically as variable
  • added order key to schema
  • order schema by order and name field

Base automatically changed from manual_testing to main October 31, 2024 12:27
@YooSunYoung YooSunYoung mentioned this pull request Nov 1, 2024
Closed
@YooSunYoung YooSunYoung merged commit 344362e into main Nov 1, 2024
3 checks passed
@YooSunYoung YooSunYoung deleted the fix_for_testing branch November 1, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YooSunYoung YooSunYoung YooSunYoung 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.
2 participants
@nitrosx @YooSunYoung