Revert "Merge pull request #197 from percona/PT-115"

This reverts commit 0f5ce4cc0b, reversing changes made to 515833f3d2.
2025-09-10 13:11:32 +00:00 · 2017-03-29 15:10:25 -03:00
parent 0f5ce4cc0b
commit c554e98926
26 changed files with 26 additions and 130 deletions
--- a/bin/pt-align
+++ b/bin/pt-align
@@ -677,11 +677,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-archiver
+++ b/bin/pt-archiver
@@ -1406,11 +1406,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-config-diff
+++ b/bin/pt-config-diff
@@ -1404,11 +1404,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-deadlock-logger
+++ b/bin/pt-deadlock-logger
@@ -754,11 +754,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-diskstats
+++ b/bin/pt-diskstats
@@ -750,11 +750,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-duplicate-key-checker
+++ b/bin/pt-duplicate-key-checker
@@ -1741,11 +1741,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-fifo-split
+++ b/bin/pt-fifo-split
@@ -678,11 +678,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-find
+++ b/bin/pt-find
@@ -1170,11 +1170,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-fingerprint
+++ b/bin/pt-fingerprint
@@ -679,11 +679,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-fk-error-logger
+++ b/bin/pt-fk-error-logger
@@ -749,11 +749,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-heartbeat
+++ b/bin/pt-heartbeat
@@ -1515,11 +1515,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-index-usage
+++ b/bin/pt-index-usage
@@ -1331,11 +1331,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-kill
+++ b/bin/pt-kill
@@ -759,11 +759,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-online-schema-change
+++ b/bin/pt-online-schema-change
@@ -767,11 +767,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-query-digest
+++ b/bin/pt-query-digest
@@ -1999,11 +1999,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-show-grants
+++ b/bin/pt-show-grants
@@ -680,11 +680,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-slave-delay
+++ b/bin/pt-slave-delay
@@ -752,11 +752,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-slave-find
+++ b/bin/pt-slave-find
@@ -687,11 +687,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-slave-restart
+++ b/bin/pt-slave-restart
@@ -904,11 +904,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-table-checksum
+++ b/bin/pt-table-checksum
@@ -2428,11 +2428,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-table-sync
+++ b/bin/pt-table-sync
@@ -767,11 +767,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-table-usage
+++ b/bin/pt-table-usage
@@ -1772,11 +1772,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-upgrade
+++ b/bin/pt-upgrade
@@ -1996,11 +1996,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-variable-advisor
+++ b/bin/pt-variable-advisor
@@ -756,11 +756,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/bin/pt-visual-explain
+++ b/bin/pt-visual-explain
@@ -1353,11 +1353,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');
--- a/lib/OptionParser.pm
+++ b/lib/OptionParser.pm
@@ -838,11 +838,7 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      if (!$opt->{attributes}->{repeatable}) {
      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
-      } else {
-          push @{$opt->{value}}, $self->{DSNParser}->parse($val, $prev, $defaults);
-      }
   }
   elsif ( $val && $opt->{type} eq 'z' ) {  # type size
      PTDEBUG && _d('Parsing option', $opt->{long}, 'as a size value');