Show per-column frequent values to the analysts.

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

Show per-column frequent values to the analysts. #4672

Open cristianberneanu opened 4 years ago

cristianberneanu commented 4 years ago

Forked from https://github.com/Aircloak/aircloak/issues/4477#issuecomment-678969000

Some conditions are only allowed on frequent values, but there is no way for the analyst to see which are they.

fjab commented 4 years ago

@sebastian, as far as I understand,

pro SQL: easily accessible
pro API end-point: "cleaner", less Aircloak-tainted SQL required

Does anything speak against allowing a special SQL query, and add an API end-point later if required? Or would you really like this to be one or the other?

sebastian commented 4 years ago

I think the API endpoint is rather redundant if we have support for it using SQL. We could bake the common values into the existing data sources enumeration endpoint we have.

fjab commented 4 years ago

Ok, so unless I'm missing something here it sounds to me like SQL support is the way to go. Happy to discuss in our next roadmap meeting, too.