baobabsoluciones/cornflow

Make default schema a required argument in ExecutionEndpoint

Open

#570 opened on Nov 11, 2024

View on GitHubContributor guide
 (0 comments) (0 reactions) (1 assignee)Python (5 forks)auto 404
good first issueserver

Repository metrics

Stars
 (21 stars)
PR merge metrics
 (PR metrics pending)

Description

Is your feature request related to a problem? Please describe. In the current setup, the ExecutionEndpoint takes a default schema if one is not provided through parameters. We want to remove this default schema so that it becomes a mandatory argument. Additionally, any constants related to this default schema in orchestrator_constants should be removed. Describe the solution you'd like The schema argument, which is already passed in the current version, should now be mandatory, and the default schema should no longer be applied automatically. Additional context This update may also require changes to documentation or examples where the default schema is assumed.

Contributor guide

Research direction
Remove the default schema from ExecutionEndpoint and make the schema argument required. Update all call sites, tests, and documentation accordingly. Also remove any related constants in orchestrator constants.
Tech stack
python
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
PythonGit
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70