SubDocUpdate Parsers for FlatPostgresCollection

[suddendust]

Description

This PR implements sub-doc update parsers for flat collections.

Testing

[x] Added integration tests to validate the behaviour on PG.
[x] Added Mongo/PG compatibility test.

Checklist:

• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• Any dependent changes have been merged and published in downstream modules

[codecov]

Codecov Report

❌ Patch coverage is 86.23853% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.19%. Comparing base (cfc9c08) to head (5b7464a).

Files with missing lines	Patch %	Lines
.../parser/filter/nonjson/field/PostgresDataType.java	71.42%	3 Missing and 3 partials ⚠️
...1/vistors/PostgresFilterTypeExpressionVisitor.java	50.00%	1 Missing and 1 partial ⚠️
...postgres/update/parser/PostgresAddValueParser.java	90.00%	1 Missing and 1 partial ⚠️
...ter/PostgresInRelationalFilterParserJsonArray.java	0.00%	1 Missing ⚠️
...update/parser/PostgresAddToListIfAbsentParser.java	85.71%	0 Missing and 1 partial ⚠️
...gres/update/parser/PostgresAppendToListParser.java	85.71%	0 Missing and 1 partial ⚠️
...update/parser/PostgresRemoveAllFromListParser.java	87.50%	0 Missing and 1 partial ⚠️
...s/update/parser/PostgresUpdateOperationParser.java	50.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #279      +/-   ##
============================================
+ Coverage     81.01%   81.19%   +0.18%     
- Complexity     1446     1459      +13     
============================================
  Files           241      238       -3     
  Lines          6979     6977       -2     
  Branches        648      653       +5     
============================================
+ Hits           5654     5665      +11     
+ Misses          907      895      -12     
+ Partials        418      417       -1     

Flag	Coverage Δ
integration	81.19% <86.23%> (+0.18%)	⬆️
unit	55.41% <17.43%> (+0.41%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[suresh-prakash]

Are these changes intentional?

[suddendust]

Yes, if you look at the diff, the earlier code had List.of(SubDocumentUpdate.of("price", 999));, so I was asserting on 999. Now, it has this:

            List.of(
                SubDocumentUpdate.builder()
                    .subDocument("price")
                    .operator(UpdateOperator.ADD)
                    .subDocumentValue(
                        org.hypertrace.core.documentstore.model.subdoc.SubDocumentValue.of(5))
                    .build());

So, I am asserting on 5.

[suresh-prakash]

Would it be helpful to have it in a common class and use it for any numeric operators?

[suddendust]

I think so, we might be doing these casts at multiple places. Will refactor if this is the case.

[suddendust]

@suresh-prakash Refactored this. This touched 2 files in the query layer but I think it's okay.