Push batchSize in not respected

[meabed]

Thank you so much for the new release, it's working great with couple of minor issues:

• When adding batchSize 1 to the push handler and doing multiple rows updates fast ( in forloop for example ) the docs are passing sometimes in batchsize correctly and sometimes in more than batchsize ( incorrectly )
• Boolean fields that are used in an index, must be required, is not working as its not reading the variable from the schema path properly and throwing an error that field is not correct in the schema.

RxDB version: 13.3.0 used in ReactNative and PouchDB

Thank you so much

[pubkey]

Pls make a PR with a test case to reproduce. I cannot touch bugs that are not reproduced.

[meabed]

Pls make a PR with a test case to reproduce. I cannot touch bugs that are not reproduced.

Thank you @pubkey appreciate your reply, I will share this shortly.

[meabed]

Thank you @pubkey I have put a simple nextjs app to demo it (https://github.com/meabed/rxdb-nextjs-demo-todo):

On codesandbox ( https://codesandbox.io/s/github/meabed/rxdb-nextjs-demo-todofile=/src/components/todo.tsx ):

I have added 10 db rows ( todos ) and as you see in the screenshot, the first item fires correctly at 1 batchSize, next push is 8 docs all together.

[pubkey]

I fixed the batchSize problem so that it is respected now.
Please also make a unit test for the boolean field problem so I will work on a fix for that.

[meabed]

Thank you so much @pubkey 🙏

Yes I will re-produce the boolean field and open an issue, I tried on nextjs it was working, for react native it wasn't working, I will check if this issue still persists.

[meabed]

I have the error here ( i will post a code to reproduce it ):
And you see the schema has both index and required:

ERROR : error creating collections RxError (SC38):
Fields of type boolean that are used in an index, must be required in the schema
Given parameters: {
index:"isNew"
field:"isNew"
schema:{
  "keyCompression": false,
  "version": 1,
  "primaryKey": "_id",
  "type": "object",
  "properties": {
    "_id": {
      "type": "string",
      "maxLength": 100
    },
    "data": {
      "type": "object"
    },
    "isNew": {
      "type": "boolean",
      "default": false
    },
    "createdAt": {
      "type": "string",
      "format": "date-time",
      "maxLength": 24
    },
    "updatedAt": {
      "type": "string",
      "format": "date-time",
      "maxLength": 24
    }
  },
  "required": [
    "_id",
    "data",
    "createdAt",
    "updatedAt",
    "isNew"
  ],
  "indexes": [
    "createdAt",
    "updatedAt",
    "isNew"
  ]
}}

[pubkey]

Ah I see. Thats good enough to reproduce, I will fix.

[pubkey]

Fixed here: 75ca7c1
Are you sure that your schema version should be 1 ?
If you create a collection for the first time, start with version 0

[meabed]

Got it, I think when I used 0 I had another issue In the migration Plug-in I will verify and let you know.

Appreciate your support ❤️

[meabed]

@pubkey Would you be planning to release this fixes soon?
Thank you