This message was deleted.
# announcementsw
worried-planet-2191
01/12/2022, 2:46 PMThis message was deleted.
f
future-teacher-7046
01/12/2022, 2:49 PM
The use of inner join means only users who are part of the segment will be included. We don't actually need any of the columns in that table, just the join is enough.
o
orange-train-515
01/12/2022, 2:50 PMOh, sorry. Just noticed there was inner join used. Don't work with it that often so misread it. Thanks
orange-train-515
01/12/2022, 2:52 PMIf I understand correctly if user belonged to a certain segment at any point in time, they will be a part of the experiment?
f
future-teacher-7046
01/12/2022, 2:55 PM
yeah, we're not using the timestamp column right now. We probably should be though
o
orange-train-515
01/12/2022, 3:13 PMThat's what I am thinking. Otherwise there can be situations when the user belongs to mutually exclusive segments like "paid" and "not paid".
``````
f
future-teacher-7046
01/12/2022, 4:17 PM
I made a PR to add a WHERE clause for this - https://github.com/growthbook/growthbook/pull/212
Doesn't work when a user goes back and forth between segments (e.g. cancelling a "paid" subscription and going back to "unpaid"). We're still trying to figure out a more general solution to this problem, but the PR should help for basic cases at least.
144 Views