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 2022/12/09 13:21:43 UTC

[GitHub] [airflow] astronaut-chris opened a new issue, #28260: WasbHook get_file method requires length and offset

astronaut-chris opened a new issue, #28260:
URL: https://github.com/apache/airflow/issues/28260

   ### Apache Airflow version
   
   2.5.0
   
   ### What happened
   
   Prior to Azure provider 5.0.0, `WasbHook.get_file` only required three arguments (`file_path`, `container_name`, and `blob_name`), but now two additional parameters - `length` and `offset` - are required by the underlying `WasbHook.download` method. However the underlying [`BlobClient.download_blob` method](https://learn.microsoft.com/en-us/python/api/azure-storage-blob/azure.storage.blob.blobclient?view=azure-python#azure-storage-blob-blobclient-download-blob) has `length` and `offset` as optional.
   
   ### What you think should happen instead
   
   `length` and `offset` should continue to be optional in `WasbHook.download` (and therefore `WasbHook.get_file`).
   
   ### How to reproduce
   
   Call `WasbHook.get_file` with only the initial three arguments (`file_path`, `container_name`, and `blob_name`).
   
   ### Operating System
   
   Debian Slim
   
   ### Versions of Apache Airflow Providers
   
   `apache-airflow-providers-microsoft-azure==5.0.0`
   
   ### Deployment
   
   Astronomer
   
   ### Deployment details
   
   _No response_
   
   ### Anything else
   
   Just wanted to say I really love your issue submission form.
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


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

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

[GitHub] [airflow] potiuk commented on issue #28260: WasbHook get_file method requires length and offset

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #28260:
URL: https://github.com/apache/airflow/issues/28260#issuecomment-1344310388

   duplicate as #28231 


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

[GitHub] [airflow] potiuk closed issue #28260: WasbHook get_file method requires length and offset

Posted by GitBox <gi...@apache.org>.
potiuk closed issue #28260: WasbHook get_file method requires length and offset
URL: https://github.com/apache/airflow/issues/28260


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