meltano / sdk

Write 70% less code by using the SDK to build custom extractors and loaders that adhere to the Singer standard: https://sdk.meltano.com
https://sdk.meltano.com
Apache License 2.0
94 stars 68 forks source link

[Docs]: Document the different values the `context` object can take and under which circumstances it's expected to be null #1371

Open edgarrmondragon opened 1 year ago

edgarrmondragon commented 1 year ago

Documentation type

Explanation

Description

https://meltano.slack.com/archives/C01PKLU5D1R/p1675116342431159?thread_ts=1674850558.848759&cid=C01PKLU5D1R

On why the context is None, it seems like it's because I'm not using Stream Partitioning or Parent-Child Streams, so no context. It was a bit confusing as the get_starting_timesamp() method have the context as an argument, but if you go down the rabbit hole of all the embedded methods, you will find out that it does not required a context in order to provide the self.stream_state. Mystery solve.

stale[bot] commented 1 year ago

This has been marked as stale because it is unassigned, and has not had recent activity. It will be closed after 21 days if no further activity occurs. If this should never go stale, please add the evergreen label, or request that it be added.

edgarrmondragon commented 1 year ago

Still relevant

stale[bot] commented 2 months ago

This has been marked as stale because it is unassigned, and has not had recent activity. It will be closed after 21 days if no further activity occurs. If this should never go stale, please add the evergreen label, or request that it be added.