We should only allow and only show segments that are based on the log_visit table.

All other segments should usually not be needed anyway to identify a data subject. And if so, it could be later still done potentially through the visitor log and segmentation. Although I doubt it is needed.

Why? Because this DB query will query over all visits, joining a segment with log_link_visit_action (or other tables) can not only take a really long time to find the data subjects, it could even result in the worst case maybe in temporary tables etc.

I think it would also make the tool slightly easier to use by showing only more relevant segments.

@mattab any chance to have this in 3.6.0?