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

Source Mixpanel: added HttpAvailabilityStrategy and undeclared fields for export and annotations streams #25056

Merged
merged 16 commits into from
May 3, 2023

Conversation

darynaishchenko
Copy link
Collaborator

@darynaishchenko darynaishchenko commented Apr 11, 2023

@darynaishchenko darynaishchenko self-assigned this Apr 11, 2023
@octavia-squidington-iii octavia-squidington-iii added area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/mixpanel labels Apr 11, 2023
@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 11, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4669033583

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 13, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4692715898

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 14, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4698461685

@darynaishchenko darynaishchenko force-pushed the daryna/source-mixpanel/24944-improvement branch from 53495cb to 0dbe389 Compare April 14, 2023 14:07
@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 14, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4700668393

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 19, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4740524252

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 19, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4742716837

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 19, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4747225657

@darynaishchenko darynaishchenko force-pushed the daryna/source-mixpanel/24944-improvement branch from 70116d0 to 6918def Compare April 21, 2023 14:31
@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 21, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4765953835

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 23, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4778018464

@darynaishchenko darynaishchenko force-pushed the daryna/source-mixpanel/24944-improvement branch from 9af7127 to a554c4f Compare April 24, 2023 08:11
@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 24, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4783992824

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 24, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4790412942

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 25, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4796231315
✅ connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4796231315
Python tests coverage:

Name                                         Stmts   Miss  Cover
----------------------------------------------------------------
source_mixpanel/utils.py                         8      0   100%
source_mixpanel/streams/revenue.py              14      0   100%
source_mixpanel/streams/funnels.py              57      0   100%
source_mixpanel/streams/cohorts.py              15      0   100%
source_mixpanel/streams/__init__.py              9      0   100%
source_mixpanel/property_transformation.py      19      0   100%
source_mixpanel/__init__.py                      2      0   100%
source_mixpanel/streams/base.py                 95      4    96%
source_mixpanel/streams/export.py               70      3    96%
source_mixpanel/streams/engage.py               88      6    93%
source_mixpanel/source.py                       79      8    90%
source_mixpanel/streams/annotations.py          16      3    81%
source_mixpanel/streams/cohort_members.py       21      7    67%
source_mixpanel/testing.py                      29     11    62%
----------------------------------------------------------------
TOTAL                                          522     42    92%

Build Passed

Test summary info:

=========================== short test summary info ============================
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:100: The previous and actual specifications are identical.
================== 41 passed, 1 skipped in 7863.43s (2:11:03) ==================

@darynaishchenko darynaishchenko changed the title Source Mixpanel: improvement Source Mixpanel: added HttpAvailabilityStrategy and undeclared fields for export and annotations streams Apr 25, 2023
Copy link
Collaborator

@lazebnyi lazebnyi left a comment

Choose a reason for hiding this comment

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

Nice!

Copy link
Contributor

@bnchrch bnchrch 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.

Before you merge would you

  1. Please make sure you update the version in both the source_definition.yaml and metadata.yaml file
  2. Run processResources

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 27, 2023

/publish connector=connectors/source-mixpanel

🕑 Publishing the following connectors:
connectors/source-mixpanel
https://github.com/airbytehq/airbyte/actions/runs/4817814234


Connector Version Did it publish? Were definitions generated?
connectors/source-mixpanel 0.1.32

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 27, 2023

/publish connector=connectors/source-mixpanel

🕑 Publishing the following connectors:
connectors/source-mixpanel
https://github.com/airbytehq/airbyte/actions/runs/4822299038


Connector Version Did it publish? Were definitions generated?

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 28, 2023

/publish connector=connectors/source-mixpanel

🕑 Publishing the following connectors:
connectors/source-mixpanel
https://github.com/airbytehq/airbyte/actions/runs/4828084680


Connector Version Did it publish? Were definitions generated?

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented Apr 28, 2023

/publish connector=connectors/source-mixpanel

🕑 Publishing the following connectors:
connectors/source-mixpanel
https://github.com/airbytehq/airbyte/actions/runs/4828199689


Connector Version Did it publish? Were definitions generated?

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented May 2, 2023

/publish connector=connectors/source-mixpanel

🕑 Publishing the following connectors:
connectors/source-mixpanel
https://github.com/airbytehq/airbyte/actions/runs/4858748155


Connector Version Did it publish? Were definitions generated?
connectors/source-mixpanel 0.1.32

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented May 2, 2023

/publish connector=connectors/source-mixpanel

🕑 Publishing the following connectors:
connectors/source-mixpanel
https://github.com/airbytehq/airbyte/actions/runs/4861561537


Connector Version Did it publish? Were definitions generated?
connectors/source-mixpanel 0.1.32

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@darynaishchenko
Copy link
Collaborator Author

/publish connector=connectors/source-mixpanel

@darynaishchenko
Copy link
Collaborator Author

darynaishchenko commented May 2, 2023

/publish connector=connectors/source-mixpanel

🕑 Publishing the following connectors:
connectors/source-mixpanel
https://github.com/airbytehq/airbyte/actions/runs/4863999124


Connector Version Did it publish? Were definitions generated?
connectors/source-mixpanel 0.1.32

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@darynaishchenko darynaishchenko merged commit 7c36b5f into master May 3, 2023
20 checks passed
@darynaishchenko darynaishchenko deleted the daryna/source-mixpanel/24944-improvement branch May 3, 2023 09:13
marcosmarxm pushed a commit to natalia-miinto/airbyte that referenced this pull request Jun 8, 2023
… for export and annotations streams (airbytehq#25056)

* added HttpAvailabilityStrategy and undeclared fields for annotations and export streams

* added extra fields to export stream, updated expected records

* added exponential backoff

* changed testing range days to 10

* updated expected records, added typeTransformer to exports stream

* added changelog

* removed HttpAvailabilityStrategy property

* added null in type for annotations schema

* removed unused imports in base.py

* bump version in metadata.yaml

* updated expected records

* auto-bump connector version

---------

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/mixpanel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Source Mixpanel: Add undeclared columns to spec Source source-mixpanel: implement AvailabilityStrategy
5 participants