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.

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

[NA] Simplify data ingestion for dataset and experiment items #191

Conversation

thiagohora
Copy link
Contributor

Details

Issues

Resolves #

Testing

Documentation

@thiagohora thiagohora requested a review from a team as a code owner September 6, 2024 13:35
@thiagohora thiagohora changed the title Thiagohora/simplify data ingestion for dataset and experiment items [NA] Simplify data ingestion for dataset and experiment items Sep 6, 2024
Copy link

@LifeXplorer LifeXplorer left a comment

Choose a reason for hiding this comment

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

Looks good!

@thiagohora thiagohora self-assigned this Sep 6, 2024
<if(item.hasNext)>
UNION ALL
,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Much nicer like this since multiple rows are allowed in a single statement:

@andrescrz andrescrz merged commit 7ab906b into main Sep 6, 2024
2 checks passed
@andrescrz andrescrz deleted the thiagohora/simplify_data_ingestion_for_dataset_and_experiment_items branch September 6, 2024 14:34
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