You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/07/15 04:52:24 UTC

[GitHub] [airflow] potiuk commented on pull request #17002: Parse JSON template files by default

potiuk commented on pull request #17002:
URL: https://github.com/apache/airflow/pull/17002#issuecomment-880389374


   Agree with @uranusjr - this is backwards-incompatible change. If anything we might want to make it an optional feature that you can enable if you want, but if we would like to implement this, we should make it a generic, optional feature for a number of file types (,json, ,yml, .csv). 
   
   However, I think it's not needed - As @uranusjr  decision whether to parse a file can be easily taken by each operator independently. 


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org