felipecrv commented on code in PR #40075:
URL: https://github.com/apache/arrow/pull/40075#discussion_r1491396835


##########
cpp/src/arrow/filesystem/azurefs.cc:
##########
@@ -2055,6 +2035,116 @@ class AzureFileSystem::Impl {
   static constexpr auto kTimeNeededForFileOrDirectoryRename = 
std::chrono::seconds{3};
 
  public:
+  /// \pre location.container is not empty.
+  /// \pre location.path is not empty.
+  /// \pre location.path does not end with a trailing slash.
+  Status DeleteFileOnFileSystem(const DataLake::DataLakeFileSystemClient& 
adlfs_client,
+                                const AzureLocation& location, bool 
require_file_to_exist,
+                                Azure::Nullable<std::string> lease_id = {}) {

Review Comment:
   This was needed for an experiment I did with the API before, but I'm 
removing this together with the implementation of `MovePaths` for 
flat-namespace accounts.



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