potiuk commented on PR #35612: URL: https://github.com/apache/airflow/pull/35612#issuecomment-1811376531
> Sorry, late to the party. Agree on that `ObjectStoragePath` as too many letters (getting an o15h :-D) and `Path` would be great but raises a lot of dis-ambiguation with `pathlib.Path`(so should NOT be done!). My previous favority was to name it `StoragePath` because it might be not solely _Object_-Storage being used. Can be actually "any" Storage, also FileStorage, correct? As `StoragePath` we only have s9h letters and is a compromise between o15h and p2h :-D +1 to what Jens wrote. The way how Python works with making class available without namespace after importing it, ambiguity between pathlib.Path will be far to likely to cause confusion. It's rather likely that someone will still want to use Path from Pathlin in the same module. -- 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]
