Skip to content

#2061 enhance type mapping form ArcadeDB to Postgres protocol#2074

Merged
robfrank merged 4 commits into
mainfrom
fix/2061-pgsql-serialization
Mar 24, 2025
Merged

#2061 enhance type mapping form ArcadeDB to Postgres protocol#2074
robfrank merged 4 commits into
mainfrom
fix/2061-pgsql-serialization

Conversation

@robfrank

@robfrank robfrank commented Mar 20, 2025

Copy link
Copy Markdown
Collaborator

What does this PR do?

A first review and enhancement of data mapping from ArcaDB and Postgres protocol

Motivation

Arrays and json values were mapped in the wrong way

Related issues

#2061

Additional Notes

Checklist

  • I have run the build using mvn clean package command
  • My unit tests cover both failure and success scenarios

@robfrank robfrank self-assigned this Mar 20, 2025
@robfrank robfrank added this to the 25.3.2 milestone Mar 20, 2025
@codacy-production

codacy-production Bot commented Mar 20, 2025

Copy link
Copy Markdown

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-0.29% 66.52%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (a2d836e) 69230 44642 64.48%
Head commit (6136186) 69332 (+102) 44508 (-134) 64.20% (-0.29%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#2074) 227 151 66.52%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more

@lvca lvca changed the title #2061 enhance type mapping form ArcaDB to Postgres protocol #2061 enhance type mapping form ArcadeDB to Postgres protocol Mar 20, 2025
@robfrank robfrank force-pushed the fix/2061-pgsql-serialization branch from 1cec46d to 3b2c22c Compare March 20, 2025 14:42
@robfrank robfrank force-pushed the fix/2061-pgsql-serialization branch from 3b2c22c to 81e54b8 Compare March 23, 2025 13:38
@robfrank robfrank merged commit 9acd802 into main Mar 24, 2025
@robfrank robfrank deleted the fix/2061-pgsql-serialization branch June 20, 2025 07:47
mergify Bot added a commit that referenced this pull request Apr 20, 2026
…n /studio [skip ci]

Bumps [postcss](https://github.com/postcss/postcss) from 8.5.9 to 8.5.10.
Release notes

*Sourced from [postcss's releases](https://github.com/postcss/postcss/releases).*

> 8.5.10
> ------
>
> * Fixed XSS via unescaped `</style>` in non-bundler cases (by [`@​TharVid`](https://github.com/TharVid)).


Changelog

*Sourced from [postcss's changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md).*

> 8.5.10
> ------
>
> * Fixed XSS via unescaped `</style>` in non-bundler cases (by [`@​TharVid`](https://github.com/TharVid)).


Commits

* [`33b9790`](postcss/postcss@33b9790) Release 8.5.10 version
* [`536c79e`](postcss/postcss@536c79e) Escape </style> in CSS output ([#2074](https://redirect.github.com/postcss/postcss/issues/2074))
* [`afa96b2`](postcss/postcss@afa96b2) Update dependencies ([#2073](https://redirect.github.com/postcss/postcss/issues/2073))
* [`effe88b`](postcss/postcss@effe88b) Typo ([#2072](https://redirect.github.com/postcss/postcss/issues/2072))
* [`3ee79a2`](postcss/postcss@3ee79a2) Thread model ([#2071](https://redirect.github.com/postcss/postcss/issues/2071))
* [`2e0683d`](postcss/postcss@2e0683d) Create incident response docs ([#2070](https://redirect.github.com/postcss/postcss/issues/2070))
* See full diff in [compare view](postcss/postcss@8.5.9...8.5.10)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=postcss&package-manager=npm\_and\_yarn&previous-version=8.5.9&new-version=8.5.10)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
Dependabot commands and options
  
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot show  ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
mergify Bot added a commit that referenced this pull request Jun 7, 2026
…5 [skip ci]

Bumps [org.jacoco:jacoco-maven-plugin](https://github.com/jacoco/jacoco) from 0.8.14 to 0.8.15.
Release notes

*Sourced from [org.jacoco:jacoco-maven-plugin's releases](https://github.com/jacoco/jacoco/releases).*

> 0.8.15
> ------
>
> New Features
> ------------
>
> * JaCoCo now officially supports Java 26 (GitHub [#2076](https://redirect.github.com/jacoco/jacoco/issues/2076)).
> * Experimental support for Java 27 class files (GitHub [#2004](https://redirect.github.com/jacoco/jacoco/issues/2004)).
> * Compatibility methods generated by Kotlin compiler for functions defined in interfaces are filtered out during generation of report (GitHub [#1905](https://redirect.github.com/jacoco/jacoco/issues/1905)).
> * Compatibility methods generated by Kotlin compiler for exposed boxed inline value classes (JvmExposeBoxed annotation) are filtered out during generation of report (GitHub [#1944](https://redirect.github.com/jacoco/jacoco/issues/1944)).
> * Methods generated by the Kotlin compiler for functions with JvmStatic annotation are filtered out during generation of report (GitHub [#2097](https://redirect.github.com/jacoco/jacoco/issues/2097)).
> * Improved filtering of bytecode generated by Kotlin compiler for when expressions and statements with kotlin.String subject where first branch condition contains string with largest hash (GitHub [#2098](https://redirect.github.com/jacoco/jacoco/issues/2098)).
> * Part of bytecode that javac versions from 24 to 26 generate for switch statements and expressions with selector expression of type java.lang.String inside lambdas is filtered out during generation of report (GitHub [#2023](https://redirect.github.com/jacoco/jacoco/issues/2023)).
> * Improved performance of Kotlin files analysis by parsing SMAPs only once per class (GitHub [#2114](https://redirect.github.com/jacoco/jacoco/issues/2114)).
> * For better performance agent output methods tcpclient and tcpserver use BufferedOutputStream to write execution data to socket. Maven plugin, Ant tasks, CLI, API usage examples, and ExecDumpClient API use BufferedInputStream to read execution data from socket. Third-party integrations should do the same to benefit from this change in agent (GitHub [#2089](https://redirect.github.com/jacoco/jacoco/issues/2089)).
>
> Fixed bugs
> ----------
>
> * Fixed processing of Kotlin SMAP in synthetic classes (GitHub [#1985](https://redirect.github.com/jacoco/jacoco/issues/1985)).
> * Multiple JaCoCo runtimes within one JVM writing to the same output file should not cause data corruption when running on JDK versions from 6 to 10 affected by [JDK-8166253](https://bugs.openjdk.org/browse/JDK-8166253) (GitHub [#2065](https://redirect.github.com/jacoco/jacoco/issues/2065), [#2074](https://redirect.github.com/jacoco/jacoco/issues/2074)).
> * For better performance agent writes to output file via BufferedOutputStream, this fixes regression introduced in version 0.6.2 (GitHub [#2073](https://redirect.github.com/jacoco/jacoco/issues/2073)).
> * Fixed NullPointerException when JaCoCo agent is loaded by non system class loader, for example when loaded by JBoss Modules (GitHub [#1651](https://redirect.github.com/jacoco/jacoco/issues/1651)).
>
> Non-functional Changes
> ----------------------
>
> * JaCoCo now depends on ASM 9.10.1 (GitHub [#2134](https://redirect.github.com/jacoco/jacoco/issues/2134)).


Commits

* [`6c5260a`](jacoco/jacoco@6c5260a) Prepare release v0.8.15
* [`5c05141`](jacoco/jacoco@5c05141) Transfer of execution data through socket should use buffered stream ([#2089](https://redirect.github.com/jacoco/jacoco/issues/2089))
* [`ab5efa9`](jacoco/jacoco@ab5efa9) Remove from Azure Pipelines all builds except with JDK 5 and JDK EA ([#2148](https://redirect.github.com/jacoco/jacoco/issues/2148))
* [`5f6ea38`](jacoco/jacoco@5f6ea38) Use Windows 2025 image in GitHub Actions ([#2130](https://redirect.github.com/jacoco/jacoco/issues/2130))
* [`35a8af2`](jacoco/jacoco@35a8af2) Use Renovate instead of Dependabot for updates of ASM ([#2137](https://redirect.github.com/jacoco/jacoco/issues/2137))
* [`85b8ddf`](jacoco/jacoco@85b8ddf) Upgrade ASM to 9.10.1 ([#2134](https://redirect.github.com/jacoco/jacoco/issues/2134))
* [`2988647`](jacoco/jacoco@2988647) AgentModule should use ClassLoader of agent instead of SystemClassLoader ([#1651](https://redirect.github.com/jacoco/jacoco/issues/1651))
* [`75a4e31`](jacoco/jacoco@75a4e31) Add filter for Kotlin `@JvmExposeBoxed` ([#1944](https://redirect.github.com/jacoco/jacoco/issues/1944))
* [`691fa1d`](jacoco/jacoco@691fa1d) Use Renovate instead of Dependabot for updates of GitHub Actions ([#2132](https://redirect.github.com/jacoco/jacoco/issues/2132))
* [`3e18f17`](jacoco/jacoco@3e18f17) Require at least JDK 21 for build ([#2128](https://redirect.github.com/jacoco/jacoco/issues/2128))
* Additional commits viewable in [compare view](jacoco/jacoco@v0.8.14...v0.8.15)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=org.jacoco:jacoco-maven-plugin&package-manager=maven&previous-version=0.8.14&new-version=0.8.15)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
Dependabot commands and options
  
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot show  ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants