You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@accumulo.apache.org by "Eric Newton (JIRA)" <ji...@apache.org> on 2012/08/29 03:22:08 UTC

[jira] [Created] (ACCUMULO-745) lazy command-line parsing

Eric Newton created ACCUMULO-745:
------------------------------------

             Summary: lazy command-line parsing
                 Key: ACCUMULO-745
                 URL: https://issues.apache.org/jira/browse/ACCUMULO-745
             Project: Accumulo
          Issue Type: Bug
          Components: client
    Affects Versions: 1.4.1
            Reporter: Eric Newton
            Assignee: Billie Rinaldi
            Priority: Trivial


In a few utilities, the command-line parsing is pretty lazy.  There's no usage, just an NPE if you don't provide all the magic options on the command-line.  In particular, Initialize doesn't use an off-the-shelf library for command-line parsing, and it really should.

See ACCUMULO-744.

In addition, many command-line utilities can and should be able to read accumulo-site.xml, and do not need to provide username/password/instance/zookeeper information by default.


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (ACCUMULO-745) Use the Apache CLI library to parse parameters in all utilities.

Posted by "Josh Elser (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ACCUMULO-745?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13443762#comment-13443762 ] 

Josh Elser commented on ACCUMULO-745:
-------------------------------------

Individual tickets seems excessive to me. Said list for reference:

{noformat}
% find server -type f -name "*.java" | xargs fgrep "public static void main" | fgrep -v 'src/test' | awk '{print $1}'
server/src/main/java/org/apache/accumulo/server/monitor/Monitor.java:
server/src/main/java/org/apache/accumulo/server/trace/TraceServer.java:
server/src/main/java/org/apache/accumulo/server/util/ListInstances.java:
server/src/main/java/org/apache/accumulo/server/util/RestoreZookeeper.java:
server/src/main/java/org/apache/accumulo/server/util/DumpZookeeper.java:
server/src/main/java/org/apache/accumulo/server/util/CheckForMetadataProblems.java:
server/src/main/java/org/apache/accumulo/server/util/VerifyTabletAssignments.java:
server/src/main/java/org/apache/accumulo/server/util/CountRowKeys.java:
server/src/main/java/org/apache/accumulo/server/util/RandomWriter.java:
server/src/main/java/org/apache/accumulo/server/util/OfflineMetadataScanner.java:
server/src/main/java/org/apache/accumulo/server/util/Initialize.java:
server/src/main/java/org/apache/accumulo/server/util/TabletServerLocks.java:
server/src/main/java/org/apache/accumulo/server/util/SendLogToChainsaw.java:
server/src/main/java/org/apache/accumulo/server/util/AddFilesWithMissingEntries.java:
server/src/main/java/org/apache/accumulo/server/util/ChangeSecret.java:
server/src/main/java/org/apache/accumulo/server/util/CleanZookeeper.java:
server/src/main/java/org/apache/accumulo/server/util/DeleteZooInstance.java:
server/src/main/java/org/apache/accumulo/server/util/Admin.java:
server/src/main/java/org/apache/accumulo/server/util/LocalityCheck.java:
server/src/main/java/org/apache/accumulo/server/util/ZooZap.java:
server/src/main/java/org/apache/accumulo/server/util/RemoveEntriesForMissingFiles.java:
server/src/main/java/org/apache/accumulo/server/util/TableDiskUsage.java:
server/src/main/java/org/apache/accumulo/server/tabletserver/LargestFirstMemoryManager.java:
server/src/main/java/org/apache/accumulo/server/tabletserver/EncodedBinaryTreePath.java:
server/src/main/java/org/apache/accumulo/server/tabletserver/TabletServer.java:
server/src/main/java/org/apache/accumulo/server/gc/SimpleGarbageCollector.java:
server/src/main/java/org/apache/accumulo/server/fate/Admin.java:
server/src/main/java/org/apache/accumulo/server/logger/LogReader.java:
server/src/main/java/org/apache/accumulo/server/problems/ProblemReports.java:
server/src/main/java/org/apache/accumulo/server/client/HdfsZooInstance.java:
server/src/main/java/org/apache/accumulo/server/test/CreateRFiles.java:
server/src/main/java/org/apache/accumulo/server/test/NativeMapConcurrencyTest.java:
server/src/main/java/org/apache/accumulo/server/test/TestIngest.java:
server/src/main/java/org/apache/accumulo/server/test/TestIngest.java:
server/src/main/java/org/apache/accumulo/server/test/WrongTabletTest.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousBatchWalker.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousVerify.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/TimeBinner.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousMoru.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/PrintScanTimeHistogram.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousQuery.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousWalk.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousScanner.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousIngest.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/GenSplits.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/UndefinedAnalyzer.java:
server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousStatsCollector.java:
server/src/main/java/org/apache/accumulo/server/test/TestMultiTableIngest.java:
server/src/main/java/org/apache/accumulo/server/test/GCLotsOfCandidatesTest.java:
server/src/main/java/org/apache/accumulo/server/test/TestBinaryRows.java:
server/src/main/java/org/apache/accumulo/server/test/QueryMetadataTable.java:
server/src/main/java/org/apache/accumulo/server/test/ListTables.java:
server/src/main/java/org/apache/accumulo/server/test/NativeMapStressTest.java:
server/src/main/java/org/apache/accumulo/server/test/EstimateInMemMapOverhead.java:
server/src/main/java/org/apache/accumulo/server/test/BulkImportDirectory.java:
server/src/main/java/org/apache/accumulo/server/test/performance/scan/CollectTabletStats.java:
server/src/main/java/org/apache/accumulo/server/test/performance/thrift/NullTserver.java:
server/src/main/java/org/apache/accumulo/server/test/performance/metadata/MetadataBatchScanTest.java:
server/src/main/java/org/apache/accumulo/server/test/CreateTestTable.java:
server/src/main/java/org/apache/accumulo/server/test/scalability/Run.java:
server/src/main/java/org/apache/accumulo/server/test/randomwalk/Framework.java:
server/src/main/java/org/apache/accumulo/server/test/randomwalk/bulk/Verify.java:
server/src/main/java/org/apache/accumulo/server/test/functional/FunctionalTest.java:
server/src/main/java/org/apache/accumulo/server/test/functional/NativeMapTest.java:
server/src/main/java/org/apache/accumulo/server/test/functional/RunTests.java:
server/src/main/java/org/apache/accumulo/server/test/functional/CacheTestClean.java:
server/src/main/java/org/apache/accumulo/server/test/functional/CacheTestReader.java:
server/src/main/java/org/apache/accumulo/server/test/functional/CacheTestWriter.java:
server/src/main/java/org/apache/accumulo/server/test/functional/ZombieTServer.java:
server/src/main/java/org/apache/accumulo/server/test/NativeMapPerformanceTest.java:
server/src/main/java/org/apache/accumulo/server/test/VerifyIngest.java:
server/src/main/java/org/apache/accumulo/server/test/GetMasterStats.java:
server/src/main/java/org/apache/accumulo/server/test/CreateRandomRFile.java:
server/src/main/java/org/apache/accumulo/server/zookeeper/ZooQueueLock.java:
server/src/main/java/org/apache/accumulo/server/zookeeper/ZooLock.java:
server/src/main/java/org/apache/accumulo/server/master/state/SetGoalState.java:
server/src/main/java/org/apache/accumulo/server/master/state/MergeStats.java:
server/src/main/java/org/apache/accumulo/server/master/Master.java:
server/src/main/java/org/apache/accumulo/server/conf/ConfigSanityCheck.java:
server/src/main/java/org/apache/accumulo/server/metrics/MetricsConfiguration.java:
server/src/main/java/org/apache/accumulo/server/metanalysis/PrintEvents.java:
server/src/main/java/org/apache/accumulo/server/metanalysis/FilterMeta.java:
server/src/main/java/org/apache/accumulo/server/metanalysis/IndexMeta.java:
server/src/main/java/org/apache/accumulo/server/metanalysis/FindTablet.java:
{noformat}
                
> Use the Apache CLI library to parse parameters in all utilities.
> ----------------------------------------------------------------
>
>                 Key: ACCUMULO-745
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-745
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.1
>            Reporter: Eric Newton
>            Assignee: Billie Rinaldi
>            Priority: Trivial
>              Labels: newbie
>
> In a few utilities, the command-line parsing is pretty lazy.  There's no usage, just an NPE if you don't provide all the magic options on the command-line.  In particular, Initialize doesn't use an off-the-shelf library for command-line parsing, and it really should.
> See ACCUMULO-744.
> In addition, many command-line utilities can and should be able to read accumulo-site.xml, and do not need to provide username/password/instance/zookeeper information by default.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Assigned] (ACCUMULO-745) Use the Apache CLI library to parse parameters in all utilities.

Posted by "Christopher Tubbs (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ACCUMULO-745?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christopher Tubbs reassigned ACCUMULO-745:
------------------------------------------

    Assignee: Christopher Tubbs  (was: Billie Rinaldi)
    
> Use the Apache CLI library to parse parameters in all utilities.
> ----------------------------------------------------------------
>
>                 Key: ACCUMULO-745
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-745
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.1
>            Reporter: Eric Newton
>            Assignee: Christopher Tubbs
>            Priority: Trivial
>              Labels: newbie
>             Fix For: 1.5.0
>
>
> In a few utilities, the command-line parsing is pretty lazy.  There's no usage, just an NPE if you don't provide all the magic options on the command-line.  In particular, Initialize doesn't use an off-the-shelf library for command-line parsing, and it really should.
> See ACCUMULO-744.
> In addition, many command-line utilities can and should be able to read accumulo-site.xml, and do not need to provide username/password/instance/zookeeper information by default.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (ACCUMULO-745) Use the Apache CLI library to parse parameters in all utilities.

Posted by "Eric Newton (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ACCUMULO-745?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13443749#comment-13443749 ] 

Eric Newton commented on ACCUMULO-745:
--------------------------------------

Everything with {{public static void main(String[] args)}} function in the server package should (optionally) use accumulo-site.xml, provide usage and use an off-the-shelf argument parsing utility, unless it is in the test directory.

Every example should also provide usage and use an off-the-shelf argument parsing utility.


                
> Use the Apache CLI library to parse parameters in all utilities.
> ----------------------------------------------------------------
>
>                 Key: ACCUMULO-745
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-745
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.1
>            Reporter: Eric Newton
>            Assignee: Billie Rinaldi
>            Priority: Trivial
>
> In a few utilities, the command-line parsing is pretty lazy.  There's no usage, just an NPE if you don't provide all the magic options on the command-line.  In particular, Initialize doesn't use an off-the-shelf library for command-line parsing, and it really should.
> See ACCUMULO-744.
> In addition, many command-line utilities can and should be able to read accumulo-site.xml, and do not need to provide username/password/instance/zookeeper information by default.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (ACCUMULO-745) Use the Apache CLI library to parse parameters in all utilities.

Posted by "Josh Elser (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ACCUMULO-745?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Josh Elser updated ACCUMULO-745:
--------------------------------

    Labels: newbie  (was: )
    
> Use the Apache CLI library to parse parameters in all utilities.
> ----------------------------------------------------------------
>
>                 Key: ACCUMULO-745
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-745
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.1
>            Reporter: Eric Newton
>            Assignee: Billie Rinaldi
>            Priority: Trivial
>              Labels: newbie
>
> In a few utilities, the command-line parsing is pretty lazy.  There's no usage, just an NPE if you don't provide all the magic options on the command-line.  In particular, Initialize doesn't use an off-the-shelf library for command-line parsing, and it really should.
> See ACCUMULO-744.
> In addition, many command-line utilities can and should be able to read accumulo-site.xml, and do not need to provide username/password/instance/zookeeper information by default.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (ACCUMULO-745) Use the Apache CLI library to parse parameters in all utilities.

Posted by "Christopher Tubbs (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ACCUMULO-745?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christopher Tubbs updated ACCUMULO-745:
---------------------------------------

    Fix Version/s: 1.5.0
    
> Use the Apache CLI library to parse parameters in all utilities.
> ----------------------------------------------------------------
>
>                 Key: ACCUMULO-745
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-745
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.1
>            Reporter: Eric Newton
>            Assignee: Billie Rinaldi
>            Priority: Trivial
>              Labels: newbie
>             Fix For: 1.5.0
>
>
> In a few utilities, the command-line parsing is pretty lazy.  There's no usage, just an NPE if you don't provide all the magic options on the command-line.  In particular, Initialize doesn't use an off-the-shelf library for command-line parsing, and it really should.
> See ACCUMULO-744.
> In addition, many command-line utilities can and should be able to read accumulo-site.xml, and do not need to provide username/password/instance/zookeeper information by default.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (ACCUMULO-745) Use the Apache CLI library to parse parameters in all utilities.

Posted by "David Medinets (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ACCUMULO-745?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Medinets updated ACCUMULO-745:
------------------------------------

    Summary: Use the Apache CLI library to parse parameters in all utilities.  (was: lazy command-line parsing)
    
> Use the Apache CLI library to parse parameters in all utilities.
> ----------------------------------------------------------------
>
>                 Key: ACCUMULO-745
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-745
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.1
>            Reporter: Eric Newton
>            Assignee: Billie Rinaldi
>            Priority: Trivial
>
> In a few utilities, the command-line parsing is pretty lazy.  There's no usage, just an NPE if you don't provide all the magic options on the command-line.  In particular, Initialize doesn't use an off-the-shelf library for command-line parsing, and it really should.
> See ACCUMULO-744.
> In addition, many command-line utilities can and should be able to read accumulo-site.xml, and do not need to provide username/password/instance/zookeeper information by default.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (ACCUMULO-745) lazy command-line parsing

Posted by "David Medinets (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ACCUMULO-745?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13443735#comment-13443735 ] 

David Medinets commented on ACCUMULO-745:
-----------------------------------------

Please be more specific. Which utilities should read accumulo-site.xml? We should probably have one ticket for each to minimize the changes related to each.
                
> lazy command-line parsing
> -------------------------
>
>                 Key: ACCUMULO-745
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-745
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.1
>            Reporter: Eric Newton
>            Assignee: Billie Rinaldi
>            Priority: Trivial
>
> In a few utilities, the command-line parsing is pretty lazy.  There's no usage, just an NPE if you don't provide all the magic options on the command-line.  In particular, Initialize doesn't use an off-the-shelf library for command-line parsing, and it really should.
> See ACCUMULO-744.
> In addition, many command-line utilities can and should be able to read accumulo-site.xml, and do not need to provide username/password/instance/zookeeper information by default.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira