Sheridan C Rawlins comments

Results 97 comments of


                                            Sheridan C Rawlins

Provide consumer DAGs with context about the dag run(s) of its producer(s)

It seems like - I could be wrong - `task_id` could be enough to get the exact xcom information. So… if a `TaskInstance` is able to know how it was...

Provide consumer DAGs with context about the dag run(s) of its producer(s)

Just thinking out loud, but if `TaskInstance` could have a copy of the schedule from its dag and a field (say `outlet_task_id` added to the Dataset class and populated when...

Please add the ability to reuse a state machine

If you're feeling really creative, you could make an interface for a state machine instance and keep a pointer to the machine and just the state can be copied to...

Please add the ability to reuse a state machine

Another idea - is to provide a very fast reset call and use sync.Pool to "reuse"

Please add the ability to reuse a state machine

Well I ended up creating a Reset trigger and stuffing into a sync.Pool for reuse. That works fine. Except that it turns the diagram into a loop. I'd love to...

update http.Client.go

Hey… this PR is a little stale and I see that at least one build failed… can you check on the test failures and update?

## Code Review Summary I've completed a comprehensive review using specialized code-reviewer and golang-pro agents. ### 🚨 Critical Issues (Must Fix) **1. Missing Media Type Validation** (`adapters/risemediatech/risemediatech.go:29-44`) - No check...

Sheridan C Rawlins

Provide consumer DAGs with context about the dag run(s) of its producer(s)

Provide consumer DAGs with context about the dag run(s) of its producer(s)

Please add the ability to reuse a state machine

Please add the ability to reuse a state machine

Please add the ability to reuse a state machine

update http.Client.go

New Adapter: RiseMediaTech