Open zh4ngx opened 3 months ago
Thanks for reporting the issueĀ @zh4ngx Railz is a community connector and it isnāt in the current roadmap for improvements. If you want to contribute fixing the issue please reach me out in Slack so I can provide you instructions to make the contribution.
Thanks @marcosmarxm . Quick question: can NestedStateCartesianProductStreamSlicer
be migrated to the low-code framework as #22873 states? Are there instructions on how to do that?
I found the issue: https://github.com/airbytehq/airbyte/commit/a45a1e33413eacccb2b8a83450d1638c48fd2042
The API was updated here but stream_slices()
on this source still uses the old API with stream_state
and sync_mode
. Will send in PR.
Adding this to our community board ā we'll take a look. I see you're interested, so I'll ask folks to post their journey. I think you're right that we should replace the whole custom component with a declarative approach.
Much appreciated. Been struggling with getting this working and it's a serious blocker for our pipeline
Connector Name
source-railz
Connector Version
0.1.3
What step the error happened?
During the sync
Relevant information
When running a source-railz sync, base syncs succeed, but incremental syncs using
base_incremental_service_stream
fail. When looking into the logs (attached below), it looks likeNestedStateCartesianProductStreamSlicer.stream_slices(sync_mode, stream_state)
is the cause.NestedStateCartesianProductStreamSlicer subclasses CartesianProductStreamSlicer, which takes in
stream_slicers: List[PartitionRouter]
. In the yaml file, 2 stream slicers are passed in:SubstreamPartitionRouter
, which subclassesPartitionRouter
, andDatetimeBasedCursor
, which does not. Both classes implement the.stream_slices()
method.When incremental sync is run, it looks like it is using
SimpleRetriever.stream_slicer.stream_slices()
(no arguments), which fails sinceNestedStateCartesianProductStreamSlicer.stream_slices(...)
requires 2 arguments.2 things pop out to me:
SimpleRetriever
expects a consistentStreamSlicer.stream_slices()
method signatureNestedStateCartesianProductStreamSlicer
is introducing 2 new method arguments toStreamSlicer
that aren't supported elsewhere.Relevant log output
Contribute