Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 166
    • Merge requests 166
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • atlasatlas
  • athenaathena
  • Merge requests
  • !60136

Support both 'flags' and 'ConfigFlags' in pre/postExec

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Tadej Novak requested to merge tadej/athena:flags-prepostexec into master Jan 26, 2023
  • Overview 4
  • Commits 1
  • Pipelines 1
  • Changes 3

Support both flags and ConfigFlags in pre/postExec. The motivation for this is that we're trying to achieve flags as de-facto variable for non-global configuration flags instance. Keep ConfigFlags for backwards compatibility for now.

Also add tests for that to ensure it actually works as expected. More tests for TransformUtils will be added separately.

/cc @fwinkl @jchapman

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: flags-prepostexec