HarshitGupta11 commented on code in PR #7197: URL: https://github.com/apache/hadoop/pull/7197#discussion_r1912716422
########## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/BulkDeleteCommand.java: ########## @@ -1,84 +1,207 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + package org.apache.hadoop.fs.shell; +import java.io.BufferedReader; +import java.io.IOException; +import java.io.InputStreamReader; +import java.nio.charset.StandardCharsets; +import java.util.*; +import java.util.stream.Collectors; + import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.BulkDelete; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import java.io.BufferedReader; -import java.io.IOException; -import java.io.InputStreamReader; -import java.util.ArrayList; -import java.util.LinkedList; -import java.util.List; -import java.util.stream.Collectors; - public class BulkDeleteCommand extends FsCommand { - public static void registerCommands(CommandFactory factory) { - factory.addClass(BulkDeleteCommand.class, "-bulkDelete"); - } - public static final String name = "bulkDelete"; + public static void registerCommands(CommandFactory factory) { + factory.addClass(BulkDeleteCommand.class, "-bulkDelete"); + } - public static final String READ_FROM_FILE = "readFromFile"; + private static final Logger LOG = LoggerFactory.getLogger(BulkDeleteCommand.class.getName()); - public static final String USAGE = "-[ " + READ_FROM_FILE + "] [<file>] [<basePath> <paths>]"; + public static final String NAME = "bulkDelete"; - public static final String DESCRIPTION = "Deletes the set of files under the given path. If a list of paths " + - "is provided then the paths are deleted directly. User can also point to the file where the paths are" + - "listed as full object names."; + /** + * File Name parameter to be specified at command line. + */ + public static final String READ_FROM_FILE = "readFromFile"; - private String fileName; + /** + * Page size parameter specified at command line. + */ + public static final String PAGE_SIZE = "pageSize"; - /* - Making the class stateful as the PathData initialization for all args is not needed - */ - LinkedList<String> childArgs; - protected BulkDeleteCommand() { - this.childArgs = new LinkedList<>(); + public static final String USAGE = "-[ " + READ_FROM_FILE + "] [<file>] [" + PAGE_SIZE + + "] [<pageSize>] [<basePath> <paths>]"; + + public static final String DESCRIPTION = "Deletes the set of files under the given <path>.\n" + + "If a list of paths is provided at command line then the paths are deleted directly.\n" + + "User can also point to the file where the paths are listed as full object names using the \"fileName\"" + + "parameter. The presence of a file name takes precedence over the list of objects.\n" + + "Page size refers to the size of each bulk delete batch." + + "Users can specify the page size using \"pageSize\" command parameter." + + "Default value is 1.\n"; + + private String fileName; + + private int pageSize; + + /** + * Making the class stateful as the PathData initialization for all args is not needed Review Comment: done -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
