[orc] branch main updated: ORC-1133: [C++] Fix csv-import tool options

[gangwu]

This is an automated email from the ASF dual-hosted git repository.

gangwu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/orc.git


The following commit(s) were added to refs/heads/main by this push:
     new 2125981  ORC-1133: [C++] Fix csv-import tool options
2125981 is described below

commit 21259815ae665f6a4beac29edb4fab52c2403e13
Author: Kyle <kyle.gra...@gmail.com>
AuthorDate: Fri Mar 25 17:19:03 2022 +0800

    ORC-1133: [C++] Fix csv-import tool options
    
    This closes #1069
---
 tools/src/CSVFileImport.cc | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/tools/src/CSVFileImport.cc b/tools/src/CSVFileImport.cc
index 393367b..280491b 100644
--- a/tools/src/CSVFileImport.cc
+++ b/tools/src/CSVFileImport.cc
@@ -301,7 +301,7 @@ int main(int argc, char* argv[]) {
   static struct option longOptions[] = {
     {"help", no_argument, ORC_NULLPTR, 'h'},
     {"delimiter", required_argument, ORC_NULLPTR, 'd'},
-    {"stripe", required_argument, ORC_NULLPTR, 'p'},
+    {"stripe", required_argument, ORC_NULLPTR, 's'},
     {"block", required_argument, ORC_NULLPTR, 'c'},
     {"batch", required_argument, ORC_NULLPTR, 'b'},
     {"timezone", required_argument, ORC_NULLPTR, 't'},
@@ -311,9 +311,8 @@ int main(int argc, char* argv[]) {
   int opt;
   char *tail;
   do {
-    opt = getopt_long(argc, argv, "i:o:s:b:c:p:t:h", longOptions, ORC_NULLPTR);
+    opt = getopt_long(argc, argv, "d:s:c:b:t:h", longOptions, ORC_NULLPTR);
     switch (opt) {
-      case '?':
       case 'h':
         helpFlag = true;
         opt = -1;
@@ -321,7 +320,7 @@ int main(int argc, char* argv[]) {
       case 'd':
         gDelimiter = optarg[0];
         break;
-      case 'p':
+      case 's':
         stripeSize = strtoul(optarg, &tail, 10);
         if (*tail != '\0') {
           fprintf(stderr, "The --stripe parameter requires an integer 
option.\n");