Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • E Eventutilities Python
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Custom issue tracker
    • Custom issue tracker
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Terraform modules
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • repos
  • data-engineeringdata-engineering
  • Eventutilities Python
  • Merge requests
  • !69

Better organize eventutilties_python modules

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Ottomata requested to merge T336488_ga_refactor into main Jun 08, 2023
  • Overview 3
  • Commits 1
  • Pipelines 7
  • Changes 18

This commit mostly just moves some code and modules around for better organization, but there are some functional changes.

  • stream manager related code now lives in the 'stream' module.

  • stream module exports public symboles using all and init.py Users should import stream manager symbols like: from eventutilities_python.stream import stream_manager, load_config, etc.

  • Stream.close has been removed from the interface. Closing is handled by stream_manager.

  • flink.py should contain only contain Flink specific things. As such, EventStreamDescriptor has been removed (importing it would cause a circular import). This lower level APIs use stream_name and schema_version, similar to the Java interface. If we want to make EventStreamDescriptor a top level Java interface, we can. Or use and wrap java EventStream class instead.

Bug: T336488

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: T336488_ga_refactor