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

Chunk huge arrays to surpress Oracle IN() warning #4906

Merged
merged 1 commit into from
Apr 8, 2021

Conversation

nickvergessen
Copy link
Member

⚠️ Theory fix, I did not execute the code

{
  "Exception": "Doctrine\\DBAL\\Query\\QueryException",
  "Message": "More than 1000 expressions in a list are not allowed on Oracle.",
  "Code": 0,
  "Trace": [
    {
      "file": "/var/www/html/apps/mail/lib/Db/MessageMapper.php",
      "line": 69,
      "function": "execute",
      "class": "OC\\DB\\QueryBuilder\\QueryBuilder",
      "type": "->"
    },
    {
      "file": "/var/www/html/apps/mail/lib/Db/MessageMapper.php",
      "line": 169,
      "function": "findUids",
      "class": "OCA\\Mail\\Db\\MessageMapper",
      "type": "->"
    },
    {
      "file": "/var/www/html/apps/mail/lib/Service/Sync/SyncService.php",
      "line": 155,
      "function": "findUidsForIds",
      "class": "OCA\\Mail\\Db\\MessageMapper",
      "type": "->"
    }

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst force-pushed the techdebt/noid/chunk-huge-arrays branch from 3d969cb to 6ad917e Compare April 8, 2021 12:00
@ChristophWurst ChristophWurst merged commit 619037b into master Apr 8, 2021
@ChristophWurst ChristophWurst deleted the techdebt/noid/chunk-huge-arrays branch April 8, 2021 12:20
@backportbot-nextcloud
Copy link

The backport to stable1.0 failed. Please do this backport manually.

@ChristophWurst
Copy link
Member

/backport to stable1.9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants