turbaszek commented on pull request #8889:
URL: https://github.com/apache/airflow/pull/8889#issuecomment-631335327


   > @mik-laj Thanks for sharing the info. `Variable` via `SecretBackend` can 
only read values and I don't think we should extend `SecretBackend` to support 
write due to security concerns.
   
   I agree with that.
   
   > Therefore, I think allowing people to provide custom `Variable` 
implementation that support read/write should fit in this use case.
   
   In general, I like the idea. However, my main concern is that with both 
variable and secret backends give users two ways of reading variables but only 
one to write them. I can imagine that:
   a) users will ask which one they should use for reading
   b) users will mix both approaches and will be confused by writing variable 
by VariableBackend and reading it by SecretBackend
   


----------------------------------------------------------------
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.

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


Reply via email to