Skip to content

Conversation

@nomadicoder
Copy link
Contributor

No description provided.

@nomadicoder nomadicoder requested a review from sensei100 May 5, 2025 18:35
- Specifies ruby version for cvs indexing
- Commented out previous tries for reference for continued development
- Stores data in /tmp for indexing
- Less generic script name
@nomadicoder nomadicoder force-pushed the DEVO-1167-Create_Airflow_DAG_for_gencon_solr_collections branch from ee6ce06 to 9092780 Compare May 5, 2025 20:35
- Move multiple tasks into the ingest script
- Force csv encoding to UTF-8
- Remove development data
- Run DAG manually
- Store raw csv data in temporary directory
@nomadicoder nomadicoder marked this pull request as ready for review May 7, 2025 19:01
@nomadicoder nomadicoder force-pushed the DEVO-1167-Create_Airflow_DAG_for_gencon_solr_collections branch 3 times, most recently from 4f6faa2 to 99e8880 Compare May 15, 2025 21:08
@nomadicoder nomadicoder marked this pull request as draft May 16, 2025 13:59
- Separate gencon_index to it's own project
- Replace literal paths and URL's to environment variables
@nomadicoder nomadicoder force-pushed the DEVO-1167-Create_Airflow_DAG_for_gencon_solr_collections branch from 99e8880 to c0af430 Compare May 16, 2025 15:31
@nomadicoder nomadicoder marked this pull request as ready for review May 16, 2025 15:35
@sensei100
Copy link
Contributor

@nomadicoder This looks mostly good, but I feel like there has to be a more Airflow-like way to add the bundle flag and the ruby version without having to create separate files for each of them. Can you investigate that?

@nomadicoder nomadicoder force-pushed the DEVO-1167-Create_Airflow_DAG_for_gencon_solr_collections branch from b2ca0e1 to c0af430 Compare June 23, 2025 21:41
@sensei100 sensei100 merged commit 26e5b8b into main Jul 23, 2025
1 check passed
@sensei100 sensei100 deleted the DEVO-1167-Create_Airflow_DAG_for_gencon_solr_collections branch July 23, 2025 14:03
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.

3 participants