pateash commented on pull request #16931:
URL: https://github.com/apache/airflow/pull/16931#issuecomment-884180567


   > The problem is we can’t store the timetable object in db (at least not 
easily; it’s technically doable but should be avoided). Maybe we can pre-render 
the description and store that (it’d be just a string) on `DagModel` instead? 
Call it `timetable_description` or something. Again, I’m not sure how best to 
do this.
   
   Honestly, I am not sure what you are trying to suggest here.
   @uranusjr, If you are trying to suggest some specific changes we should do, 
let me know.
   
   currently, timetable.description has same logic for DAG and DagModel which 
might change in future as you pointed out.
   
   We can modify that behaviour later whenever we do that change.
   
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to