-
Notifications
You must be signed in to change notification settings - Fork 237
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
Redundant InterestGroup and values #974
Comments
(For context for other readers: the screenshot you're including here is from Chrome's DevTools panel — choose the "Application" tab and then "Interest groups" from the left panel.) @anderagakura The list that appears in DevTools is a record of all Interest Group events. So each time an IG is joined or loaded, we add an entry to that list. The "Event time" column is just telling you at what time that event was logged. An IG will appear multiple times if we load it multiple times, which can happen if there are multiple auctions. If you click on one of these rows, you'll see the current value of the IG. It's only stored once, even though the log shows that we loaded it from the database multiple times. |
@michaelkleber So, your answer brings my next point which is : At high scale (I mean more IGs), would not it be a risk of memory leak for the browser during the user journey?
Of course, when I click I definitely see the value of the IG but how do we know it's only stored once if we see the IG multiple times?
|
I'm not sure what potential memory leak you are worried about.
I don't believe our tooling has a way to see the list of currently-stored IGs per owner, but @morlovich please let me know if I'm wrong. |
There is no UI for that, no, though you can peek at at the database file with sqlite. Probably best not to do it while Chrome is running. e.g. for a source build (packages would use coming like google-chrome) on Linux: And you can do stuff like: ...at any rate the interest groups table has PRIMARY KEY(owner,name). |
SQLLite hasn't permitted opening the db (with Chrome running) since it has some exclusive lock. |
@morlovich You already know but I confirm that every IG is counted. The result is 2 per IG because it's based on a new scenario on my side. Btw, I ran it on Mac |
@morlovich The data type of the column |
I don't understand your question fully, but that's in bytes if you mean that |
Just a minor thing for future IG detail searchers:
|
I'd like to understand a couple of things about the redundant
InterestGroup
and values situation :In Chrome browser,
Event time
is updated after every refresh. To be more explicit, should not it be clearer to get the field namecreated date
orupdated date
?Note : This situation seems to be only applied to the latest
InterestGroup
with the same values in a loop until the browser get a new event.When I browse or refresh the browser, the same
InterestGroup
with the same values can be added instead of updating or overwriting the existing one. Which results to get a redundant list ofInterestGroup
and values.Question : At high scale, would not it be a risk of
memory leak
for the Chrome browser during the user journey?The text was updated successfully, but these errors were encountered: