Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

issue-902 : Debug assertion 'max_val == std::numeric_limits<ulonglong… #909

Closed

Conversation

george-lorch
Copy link

…>::max()' with auto inc manually set > than field limit

  • Added extra check to see if current auto_inc value is greater than the fields
    max_value, and if so, return ullong_max instead of trying to find a new value.

  • Updated autoinc_vars test with a simple sequence that would have hit the debug
    assertion prior to the fix but now illustrates proper (well, similar to
    innodb) behavior.

…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lth has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

george-lorch pushed a commit to george-lorch/percona-server that referenced this pull request Jan 9, 2019
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
george-lorch pushed a commit to george-lorch/percona-server that referenced this pull request Jan 9, 2019
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
george-lorch pushed a commit to george-lorch/percona-server that referenced this pull request Jan 10, 2019
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
abhinav04sharma pushed a commit to abhinav04sharma/mysql-5.6 that referenced this pull request Mar 22, 2019
…limits<ulonglong… (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9
facebook-github-bot pushed a commit that referenced this pull request Dec 23, 2019
#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: #909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 3798f87
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Aug 13, 2020
facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 3798f87
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Sep 9, 2020
facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 3798f87
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Sep 16, 2020
facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 3798f87
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Oct 5, 2020
facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 3798f87
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Nov 11, 2020
facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 3798f87
facebook-github-bot pushed a commit that referenced this pull request Mar 8, 2021
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(#909)?= (#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: #909

Differential Revision: D13243938 (828629b)

Pulled By: lth

fbshipit-source-id: b0613eab185
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Aug 16, 2021
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938 (facebook@828629b)

Pulled By: lth

fbshipit-source-id: b0613eab185
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Aug 17, 2021
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938 (facebook@828629b)

Pulled By: lth

fbshipit-source-id: b0613eab185
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Aug 30, 2021
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938 (facebook@828629b)

Pulled By: lth

fbshipit-source-id: b0613eab185
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Aug 30, 2021
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938 (facebook@828629b)

Pulled By: lth

fbshipit-source-id: b0613eab185
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Aug 31, 2021
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938 (facebook@828629b)

Pulled By: lth

fbshipit-source-id: b0613eab185
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Aug 31, 2021
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938 (facebook@828629b)

Pulled By: lth

fbshipit-source-id: b0613eab185
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Sep 1, 2021
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938 (facebook@828629b)

Pulled By: lth

fbshipit-source-id: b0613eab185
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Sep 2, 2021
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938 (facebook@828629b)

Pulled By: lth

fbshipit-source-id: b0613eab185
hermanlee pushed a commit that referenced this pull request Jan 10, 2022
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(#909)?= (#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: #909

Differential Revision: D13243938

Pulled By: lth
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Jan 17, 2022
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938

Pulled By: lth
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Jan 17, 2022
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938

Pulled By: lth
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request Jan 17, 2022
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938

Pulled By: lth
ldonoso pushed a commit to ldonoso/percona-server that referenced this pull request Mar 15, 2022
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(percona#909)?= (percona#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth
inikep pushed a commit to inikep/mysql-5.6 that referenced this pull request May 20, 2022
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(facebook#909)?= (facebook#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook#909

Differential Revision: D13243938

Pulled By: lth
inikep pushed a commit to inikep/percona-server that referenced this pull request Aug 1, 2023
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
inikep pushed a commit to inikep/percona-server that referenced this pull request Aug 14, 2023
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
inikep pushed a commit to inikep/percona-server that referenced this pull request Aug 24, 2023
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
inikep pushed a commit to percona/percona-server that referenced this pull request Sep 15, 2023
…limits<ulonglong… (#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
inikep pushed a commit to inikep/percona-server that referenced this pull request Oct 16, 2023
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
inikep pushed a commit to inikep/percona-server that referenced this pull request Oct 25, 2023
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
inikep pushed a commit to inikep/percona-server that referenced this pull request Oct 26, 2023
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
inikep pushed a commit to inikep/percona-server that referenced this pull request Oct 26, 2023
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Nov 1, 2023
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Nov 10, 2023
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
inikep pushed a commit to inikep/percona-server that referenced this pull request Jan 17, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
inikep pushed a commit to inikep/percona-server that referenced this pull request Jan 17, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Jan 18, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Jan 21, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Jan 26, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Jan 30, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
inikep pushed a commit to inikep/percona-server that referenced this pull request Apr 9, 2024
…=20=3D=3D=20std::numeric=5Flimits<ulonglong=E2=80=A6=20(percona#909)?= (percona#909)

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 12, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 12, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 12, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 15, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 15, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 15, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 16, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 16, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 16, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 16, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 16, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 16, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
oleksandr-kachan pushed a commit to oleksandr-kachan/percona-server that referenced this pull request Apr 18, 2024
…limits<ulonglong… (percona#909)

Upstream commit ID : fb-mysql-5.6.35/828629b589ffb17e2ec2523ad8c2891ae9c683eb
PS-5217 : Merge fb-prod201803

Summary:
…>::max()' with auto inc manually set > than field limit

- Added extra check to see if current auto_inc value is greater than the fields
  max_value, and if so, return ullong_max instead of trying to find a new value.

- Updated autoinc_vars test with a simple sequence that would have hit the debug
  assertion prior to the fix but now illustrates proper (well, similar to
  innodb) behavior.
Pull Request resolved: facebook/mysql-5.6#909

Differential Revision: D13243938

Pulled By: lth

fbshipit-source-id: 0c9d0d9312b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants