accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-745) Use the Apache CLI library to parse parameters in all utilities.
Date Wed, 29 Aug 2012 02:37:07 GMT

    [ 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

Mime
View raw message