Sam Cohen
Sam Cohen
@eerimoq / @juleq I think I'm going to need some help from you on this. Please let me know your thoughts if you have time to take a look.
Thank you for your quick reply. I implemented both of those, and a unit test is still breaking. I tried something similar to this as well - the problem (I...
I agree, I think the root issue is that the parser naively tokenizes first using regex, without using the grammar to aid its parsing. And you're right about the priority...
> Hi @samc1213, > > I think that it is a good enhancement, especially if you are on a resource constrained system and want to get rid of the "useless"...
> @samc1213 It's been a few months since your question, but fwiw, we use enums on an internal fork. Here's a link to the commit, in case it helps you:...
Since the encode and decode functions deal only with floats, it is not always preferable to use them for all signals. We have some internal scripting that does autogenerate the...
Should the `poke` logic be shared between the `TaskStateTrigger` and the `ExternalTaskSensor`? I'm working on a change that would utilize the `poke` logic (which seems to be sound) in the...
Would anyone mind taking a look at this ( https://github.com/apache/airflow/compare/main...samc1213:airflow:external-task-deferrable ) general idea? Not ready for a deep review, but I've basically moved the `poke` logic into the `TaskStateTrigger`. From...
> > Would anyone mind taking a look at this ( [main...samc1213:airflow:external-task-deferrable](https://github.com/apache/airflow/compare/main...samc1213:airflow:external-task-deferrable) ) general idea? > > It would be nice to create PR, in this case it would be...
I'm interested in taking this issue