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

dev 241 add postgres cdc source to python sdk #34

Merged

Conversation

PabloPardoGarcia
Copy link
Member

@PabloPardoGarcia PabloPardoGarcia commented Dec 4, 2024

  • Update API models to include latest sources (postgres) and sinks (amazon_s3, snowflake)
  • Fix query parameters in API requests

Copy link

github-actions bot commented Dec 4, 2024

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py70100% 
api_client.py41392%84–86
client.py371267%98, 147, 168, 172–173, 178–181, 183, 185, 209
config.py80100% 
pipeline.py1811492%77, 87, 111, 236, 239, 246, 409–410, 412, 414, 485, 493, 509, 529
pipeline_data.py88792%52, 129, 146–147, 183, 198–199
space.py38392%109–110, 112
models
   __init__.py00100% 
models/api
   __init__.py20100% 
   api.py3240100% 
models/errors
   __init__.py30100% 
   clienterror.py35780%47–49, 51, 106–107, 119
   error.py10190%25
models/operations
   __init__.py100100% 
   access_token.py100100% 
   artifact.py100100% 
   base.py320100% 
   consumeevent.py180100% 
   consumefailed.py18194%64
   function.py280100% 
   pipeline.py500100% 
   publishevent.py140100% 
   space.py290100% 
utils
   __init__.py20100% 
   utils.py38823040%48, 53, 56–57, 61–66, 72–77, 79–80, 86–91, 93, 95–99, 103, 106, 122, 150, 153–155, 157, 161, 170–171, 176, 182, 205, 207–209, 211, 217, 219–220, 222–227, 229–231, 233–236, 238, 244, 248, 252, 255–258, 260–263, 265, 269, 271, 275, 278, 282, 284–285, 287, 321, 335, 346, 367–372, 374, 395, 397–399, 401–404, 406–407, 409–412, 414, 416, 420, 424–427, 434, 436–440, 442–444, 446, 448–449, 451–452, 463–466, 468, 470, 474, 476–477, 479, 495, 497–501, 503–505, 507–508, 510, 512–513, 515–518, 520–521, 523, 525–526, 528, 530–532, 534–537, 539, 541–542, 554–557, 559–560, 562–564, 566–568, 570–571, 573–574, 576–577, 579, 581–583, 585–586, 588–589, 591–592, 594, 596–598, 600, 602–603, 607, 615–616, 625, 639–641, 643–644, 646–647, 652, 659, 661, 675–680, 687
TOTAL138327879% 

Tests Skipped Failures Errors Time
59 0 💤 0 ❌ 0 🔥 38.881s ⏱️

ashish-bagri
ashish-bagri previously approved these changes Dec 10, 2024
@PabloPardoGarcia PabloPardoGarcia merged commit e714fe2 into main Dec 10, 2024
3 checks passed
@PabloPardoGarcia PabloPardoGarcia deleted the pablo/dev-241-add-postgres-cdc-source-to-python-sdk branch December 10, 2024 09:16
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.

2 participants