hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-15849) Shell Cleanup: Simplify handling of commands' runtime
Date Sat, 04 Jun 2016 01:52:59 GMT

    [ https://issues.apache.org/jira/browse/HBASE-15849?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15315248#comment-15315248
] 

Hudson commented on HBASE-15849:
--------------------------------

FAILURE: Integrated in HBase-1.4 #193 (See [https://builds.apache.org/job/HBase-1.4/193/])
HBASE-15849 Simplify the way we handle runtime of commands. Functions (appy: rev 1972996ffcbcbd13327c277321a687e7559991ca)
* hbase-shell/src/main/ruby/shell/commands/deleteall.rb
* hbase-shell/src/main/ruby/shell/commands/normalize.rb
* hbase-shell/src/main/ruby/shell/commands/describe.rb
* hbase-shell/src/main/ruby/shell/commands/trace.rb
* hbase-shell/src/main/ruby/shell/commands/restore_snapshot.rb
* hbase-shell/src/main/ruby/shell/commands/locate_region.rb
* hbase-shell/src/main/ruby/shell/commands/get_table.rb
* hbase-shell/src/main/ruby/shell/commands/list.rb
* hbase-shell/src/main/ruby/shell/commands/revoke.rb
* hbase-shell/src/main/ruby/shell/commands/list_snapshots.rb
* hbase-shell/src/main/ruby/shell/commands/create_namespace.rb
* hbase-shell/src/main/ruby/shell/commands/update_peer_config.rb
* hbase-shell/src/main/ruby/shell/commands/balancer.rb
* hbase-shell/src/main/ruby/shell/commands/catalogjanitor_enabled.rb
* hbase-shell/src/main/ruby/shell/commands/abort_procedure.rb
* hbase-shell/src/main/ruby/shell/commands/scan.rb
* hbase-shell/src/main/ruby/shell/commands/list_labels.rb
* hbase-shell/src/main/ruby/shell/commands/catalogjanitor_run.rb
* hbase-shell/src/main/ruby/shell/commands/wal_roll.rb
* hbase-shell/src/main/ruby/shell/commands/snapshot.rb
* hbase-shell/src/main/ruby/shell/commands/disable.rb
* hbase-shell/src/main/ruby/shell/commands/get_peer_config.rb
* hbase-shell/src/main/ruby/shell/commands/get.rb
* hbase-shell/src/main/ruby/shell/commands/alter_namespace.rb
* hbase-shell/src/main/ruby/shell/commands/put.rb
* hbase-shell/src/main/ruby/shell/commands/create.rb
* hbase-shell/src/main/ruby/shell/formatter.rb
* hbase-shell/src/main/ruby/shell/commands/catalogjanitor_switch.rb
* hbase-shell/src/main/ruby/shell/commands/is_enabled.rb
* hbase-shell/src/test/ruby/shell/shell_test.rb
* hbase-shell/src/main/ruby/shell/commands/enable.rb
* hbase-shell/src/main/ruby/shell/commands/set_auths.rb
* hbase-shell/src/main/ruby/shell/commands/drop_namespace.rb
* hbase-shell/src/main/ruby/shell/commands/show_filters.rb
* hbase-shell/src/main/ruby/shell/commands/remove_peer.rb
* hbase-shell/src/main/ruby/shell/commands/compact_rs.rb
* hbase-shell/src/main/ruby/shell/commands/add_labels.rb
* hbase-shell/src/main/ruby/shell/commands/describe_namespace.rb
* hbase-shell/src/main/ruby/shell/commands/incr.rb
* hbase-shell/src/main/ruby/shell/commands/splitormerge_switch.rb
* hbase-shell/src/main/ruby/shell/commands/normalizer_enabled.rb
* hbase-shell/src/main/ruby/shell/commands/update_config.rb
* hbase-shell/src/main/ruby/shell/commands/list_quotas.rb
* hbase-shell/src/main/ruby/shell/commands/move.rb
* hbase-shell/src/main/ruby/shell/commands/exists.rb
* hbase-shell/src/main/ruby/shell/commands/major_compact.rb
* hbase-shell/src/main/ruby/shell/commands/list_replicated_tables.rb
* hbase-shell/src/main/ruby/shell/commands/list_namespace.rb
* hbase-shell/src/main/ruby/shell/commands/balance_switch.rb
* hbase-shell/src/main/ruby/shell/commands/assign.rb
* hbase-shell/src/main/ruby/shell/commands/drop.rb
* hbase-shell/src/main/ruby/shell/commands.rb
* hbase-shell/src/main/ruby/shell/commands/delete_snapshot.rb
* hbase-shell/src/main/ruby/shell/commands/list_peers.rb
* hbase-shell/src/main/ruby/shell/commands/merge_region.rb
* hbase-shell/src/main/ruby/shell/commands/add_peer.rb
* hbase-shell/src/main/ruby/shell/commands/update_all_config.rb
* hbase-shell/src/main/ruby/shell/commands/unassign.rb
* hbase-shell/src/main/ruby/shell/commands/disable_peer.rb
* hbase-shell/src/main/ruby/shell/commands/set_visibility.rb
* hbase-shell/src/main/ruby/shell/commands/clear_auths.rb
* hbase-shell/src/main/ruby/shell/commands/balancer_enabled.rb
* hbase-shell/src/main/ruby/shell.rb
* hbase-shell/src/main/ruby/shell/commands/delete_table_snapshots.rb
* hbase-shell/src/main/ruby/shell/commands/user_permission.rb
* hbase-shell/src/main/ruby/shell/commands/list_peer_configs.rb
* hbase-shell/src/main/ruby/shell/commands/append_peer_tableCFs.rb
* hbase-shell/src/main/ruby/shell/commands/append.rb
* hbase-shell/src/main/ruby/shell/commands/is_disabled.rb
* hbase-shell/src/main/ruby/shell/commands/set_peer_tableCFs.rb
* hbase-shell/src/test/ruby/test_helper.rb
* hbase-shell/src/main/ruby/shell/commands/grant.rb
* hbase-shell/src/main/ruby/shell/commands/compact.rb
* hbase-shell/src/main/ruby/shell/commands/count.rb
* hbase-shell/src/main/ruby/shell/commands/close_region.rb
* hbase-shell/src/main/ruby/shell/commands/truncate.rb
* hbase-shell/src/main/ruby/shell/commands/list_table_snapshots.rb
* hbase-shell/src/main/ruby/shell/commands/split.rb
* hbase-shell/src/main/ruby/shell/commands/normalizer_switch.rb
* hbase-shell/src/main/ruby/shell/commands/clone_snapshot.rb
* hbase-shell/src/main/ruby/shell/commands/alter.rb
* hbase-shell/src/main/ruby/shell/commands/truncate_preserve.rb
* hbase-shell/src/test/ruby/shell/formatter_test.rb
* hbase-shell/src/main/ruby/shell/commands/delete_all_snapshot.rb
* hbase-shell/src/main/ruby/shell/commands/enable_table_replication.rb
* hbase-shell/src/main/ruby/shell/commands/flush.rb
* hbase-shell/src/main/ruby/shell/commands/list_procedures.rb
* hbase-shell/src/main/ruby/shell/commands/enable_peer.rb
* hbase-shell/src/main/ruby/shell/commands/alter_async.rb
* hbase-shell/src/main/ruby/shell/commands/delete.rb
* hbase-shell/src/main/ruby/shell/commands/splitormerge_enabled.rb
* hbase-shell/src/main/ruby/shell/commands/remove_peer_tableCFs.rb
* hbase-shell/src/main/ruby/shell/commands/get_auths.rb
* hbase-shell/src/main/ruby/shell/commands/disable_table_replication.rb
* hbase-shell/src/main/ruby/shell/commands/list_namespace_tables.rb


> Shell Cleanup: Simplify handling of commands' runtime
> -----------------------------------------------------
>
>                 Key: HBASE-15849
>                 URL: https://issues.apache.org/jira/browse/HBASE-15849
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Appy
>            Assignee: Appy
>            Priority: Minor
>             Fix For: 2.0.0
>
>         Attachments: HBASE-15849.master.001.patch
>
>
> Functions format_simple_command and format_and_return_simple_command are used to print
runtimes right now. They are called from within every single command and use Ruby's 'yield'
magic.  Instead, we can simplify it using 'command_safe' function. Since command_safe wraps
all commands, we can simply time before and after we call individual command.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message