Graphback: `sync` Query filter expression using "gt" instead of "ge"

[ssd71]

Currently, in datasync, we are using the following as our filter expression for delta queries (sync queries):
https://github.com/aerogear/graphback/blob/c0056b4bd3250df8a3e39131302dcd3138b312b9/packages/graphback-datasync/src/providers/DatasyncMongoDBDataProvider.ts#L108-L113
Would it be better if we had ge instead of gt ?
It seems to me (as far as I can understand) that gt might miss some updates that may be happening at the same time as the sync query, therefore, would it be better if we did ge(greater than or equal) instead of gt(strict inequality)?

[machi1990]

/cc @craicoverflow, @machi1990, @wtrocki

[wtrocki]

Yep. Looks like oversight

[wtrocki]

@machi1990 ROLF - now I will never know if that was you doing all nighter or you confired bot that has AI and applies labels :D

[machi1990]

@machi1990 ROLF - now I will never know if that was you doing all nighter or you confired bot that has AI and applies labels :D

It's indeed confusing. We should have a dedicated account soon.

_It's me not a bot here ;-)_

[wtrocki]

We can keep it as it fits perfectly to the joke of yourself having 24 hours triage coverage... I would extend the boot to do more actions like:

/Machi approve please

Imagine your contribution graph after that

[craicoverflow]

@wtrocki did you mean to close this?

[wtrocki]

Yep.. That was discussion issue. @ssd71 will apply this in single line on the existing PR.