Merge branch '3.0' into PT-63

2025-09-07 12:58:45 +00:00 · 2017-03-29 14:46:55 -03:00
parent 6747851dac 0f5ce4cc0b
commit 2ebd886289
26 changed files with 130 additions and 26 deletions
--- a/bin/pt-align
+++ b/bin/pt-align
@@ -677,7 +677,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +1406,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +1404,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +754,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +750,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +1741,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +678,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +1170,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +679,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +749,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +1515,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +1331,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +759,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +767,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +1999,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +680,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +752,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +687,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +904,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +2428,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +767,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +1772,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +1996,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +756,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +1353,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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,7 +838,11 @@ sub _validate_type {
         }
      }
      my $defaults = $self->{DSNParser}->parse_options($self);
-      $opt->{value} = $self->{DSNParser}->parse($val, $prev, $defaults);
+      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');