+1 (binding)
1. Verified checksums
2. Verified running server binary distribution with run.sh and
configuration overrides, feature flags per instructions under
https://polaris.apache.org/in-dev/unreleased/configuration/
On Mon, May 5, 2025 at 11:35 PM Jean-Baptiste Onofré
wrote:
> Hi Adnan
>
>
Thanks for the clarification, JB!
Packaging both the server and the admin tool in the same distribution
(archive) is certainly a good idea.
Cheers,
Dmitri.
On Tue, May 6, 2025 at 2:33 AM Jean-Baptiste Onofré wrote:
> Hi
>
> We can see two aspects here:
> - the merge of the applications
> - gat
+1 (binding)
I checked:
- checksum and signature are good on the source distribution
- in the source distribution, some files don't contain ASF header but
expected (like version.txt, etc)
- no binary file found in the source distribution
- LICENSE and NOTICE looks good in the source distribution
-
+1
1. Verified checksums, signature, artifacts
2. did manual testing with admin and server, works well !
Best,
Prashant Singh
On Tue, May 6, 2025 at 7:52 AM Jean-Baptiste Onofré wrote:
> +1 (binding)
>
> I checked:
> - checksum and signature are good on the source distribution
> - in the sourc
I'd propose to convert all existing tests that use a Postgres database to
JDBC.
Then (in the same PR) add dedicated integration tests for EclipseLink
following the same approach that was used for JDBC ITs.
This ensures that EclipseLink is covered by the normative Polaris
"integration-tests". At t
+1 (non-binding)
1. Verified signatures and checksums
2. Build and local tests all look good.
Best,
Jonas
On Tue, May 6, 2025 at 8:44 AM Prashant Singh
wrote:
> +1
>
> 1. Verified checksums, signature, artifacts
> 2. did manual testing with admin and server, works well !
>
> Best,
> Prashant S
Other than duplicated downloads for shared jars, the big problem is that
the configuration difference between admin tool and server may cause server
issues, e.g., the admin tool may bootstrap a realm in a wrong database.
Let's move them into the same distribution package first. Then we can
conside
