search

Aircloak / aircloak

This repository contains the Aircloak Air frontend as well as the code for our Cloak query and anonymization platform
2 stars 0 forks source link

Carte blanche to drop support for other things we no longer need if we need to #4894

Closed sebastian closed 3 years ago

sebastian commented 3 years ago

See this issue for an example (https://github.com/Aircloak/aircloak/issues/4893). I am having issues making a docker container since the MySQL ODBC drivers we require don't exist for recent Debian versions.

Chances are we'll hit other snags like this. I think we should decide up front that when such an event happens with ruthlessly scrap support for such features as long as they are not used by Telefonica or American Institute for Physics. Basically that means either Postgres or Oracle.

I only want to scrap features as needed. Scrapping stuff that still builds fine might cause other unintended bugs.

sebastian commented 3 years ago

@fjab: what do you think?

fjab commented 3 years ago

Yes, I think that's fine. Scrap what we don't need.

Docker isn't used by AIP?

We should also check with Clinomic once more. I'll pop him an email.

sebastian commented 3 years ago

Docker isn't used by AIP?

Oh absolutely! Everyone uses our docker containers!

fjab commented 3 years ago

Ah. And it will keep working when we drop MySQL / MariaDB.

sebastian commented 3 years ago

Ah. And it will keep working when we drop MySQL / MariaDB.

We will be able to make docker builds again, but the new docker builds would not support MariaDB or MySQL at all. So effectively our product would no longer work with those databases.

sebastian commented 3 years ago

Carte blanche seems to have been given. We might still drop Apache Impala, but things build at the moment, so we might also just as well leave it be.