Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should 'tx_falsecuredownload_download' be excluded from reference index? #243

Open
LeoniePhiline opened this issue Sep 6, 2024 · 1 comment

Comments

@LeoniePhiline
Copy link
Contributor

We have a considerable amount (almost 12 M) of records in the tx_falsecuredownload_download table, and we would want to keep them. (We do already have automated deletion of old records in place - the project is just very active.)

This makes updating or even checking the reference index take hours. Since tx_falsecuredownload_download is a statistics table, the references might not be relevant from a system perspective.

Should fal_securedownload always listen to the IsTableExcludedFromReferenceIndexEvent event and exclude the download statistics table?

This is trivial to implement per project. I merely want to start a discussion if this should be fal_securedownload's default behavior.

@FamousWolf
Copy link
Contributor

When creating a new download record, the reference index isn't updated. The reference index records are only created if you manually (or periodically) update the reference index. So excluding it from reference index shouldn't cause a problem in most cases. I wouldn't do it through IsTableExcludedFromReferenceIndexEvent though, but by changing the fields in TCA. The table is hidden in the backend, so there's no reason for the fields to be group.

However I can imagine some people using the reference index for something, so not adding these relations to the reference index might be a breaking change for them. @joey-bolts What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants