Nigel Jones

Results 462 comments of Nigel Jones
trafficstars

It appears to me that each egeria repository server should have some level of isolation in terms of it's repo - ie it should have it's 'own' xtdb repository (mostly...

Each egeria server replica (ie instances of cocoMDS2 etc) has a common configuration, and will use the same back end persistent stores, indeed this is required as all should behave...

@dwolfson You might want to retry the postgres/xtdb today. CTS test was successful last night.

Additional dependencies are likely needed - for example in your case, for using JDBC with xtdb The docs at https://egeria-project.org/connectors/repository/xtdb/?h=xtdb#pluggable-persistence have some details To use in the helm charts, you...

I noticed a few other improvements we could make, especially around consistency of dependencies, and reducing what we duplicate, but in the process hit https://github.com/odpi/egeria/issues/7421 Whilst building an uber jar...

The main fix is done now, and merged. However will leave open just to come back and refine the constraints & test

Parallelism : I believe currently these requests are single threaded in Egeria itself. If the query is across multiple repos an enterprise connector could implement this in parallel - but...

FYI am aware of the test failure. Will be working on update for that. (apologies for delay) + arranging CLA.

The test failure occurs within *test_set_ecdh_curve* which tests a follows: ``` @unittest.skipUnless(ssl.HAS_ECDH, "ECDH disabled on this OpenSSL build") def test_set_ecdh_curve(self): ctx = ssl.SSLContext(ssl.PROTOCOL_TLS_SERVER) ctx.set_ecdh_curve("prime256v1") ctx.set_ecdh_curve(b"prime256v1") self.assertRaises(TypeError, ctx.set_ecdh_curve) self.assertRaises(TypeError, ctx.set_ecdh_curve, None)...

The current PR proposal builds cleanly. However is an area that needs discussion Previously an invalid (no NID) curve would result in a value exception, whilst any other kind of...