robg-eb
robg-eb
For those struggling with this, I've found that using Meltano as a wrapper around this tap instead of just using Singer directly makes this entity/attribute selection process much easier. Instead...
@dstandish and team - I see this issue was auto-closed and now reopened. My team is struggling with the same issue, so we'd be happy to help/test in any way...
@dstandish - To clarify, is this PR ready to test as-is?
We want to keep this PR active - I can test this week as needed!
@dstandish and team - Thank you for creating this PR, as we are currently unable to properly use retries with Deferrable (Async) sensors, because of the inconsistency in retry /...
@dstandish - I'd hate to see this PR get closed as we seemed to be on the right track here , and the issue still exists around this inconsistent behavior...
@dstandish I had tested this out on Jan 29 2024 and left the feedback in the comment above - https://github.com/apache/airflow/pull/33718#issuecomment-1914788890. I see there's now a new commit as of 11...
This is still an open issue that needs to be addressed for consistency between deferrable and non-deferrable sensors.
@vahmed-hamdy - > There are some style violations, Could you run `mvn spotless:apply` ahead of the PR please? Done, I've applied updated formatting now!
I addressed all the open comments from @z3d1k now. Thank you all for the suggestions! Sounds like @hlteoh37 is the next who should take a look?