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.

Sign up for GitHub

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

Jump to bottom

only allow TTL(retention_seconds) on append only table #14791

Closed
st1page opened this issue Jan 25, 2024 · 3 comments · Fixed by #15268
Closed

only allow TTL(retention_seconds) on append only table #14791

st1page opened this issue Jan 25, 2024 · 3 comments · Fixed by #15268
Assignees
@st1page
Milestone
release-1.7

Comments

@st1page
Copy link
Contributor

st1page commented Jan 25, 2024

No description provided.

@st1page st1page self-assigned this Jan 25, 2024
@github-actions github-actions bot added this to the release-1.7 milestone Jan 25, 2024
@st1page st1page changed the title only allow TTL(retention_seconds) on append only table only allow TTL(retention_seconds) on append only table and it's indexes Jan 25, 2024
@st1page st1page changed the title only allow TTL(retention_seconds) on append only table and it's indexes only allow TTL(retention_seconds) on append only table Jan 25, 2024
@st1page st1page mentioned this issue Jan 25, 2024
Merged
9 tasks
@hzxa21
Copy link
Collaborator

hzxa21 commented Jan 29, 2024

Do we allow TTL on index of append only table?

@BugenZhao
Copy link
Member

BugenZhao commented Jan 29, 2024 •
edited
Loading

Will the consistency still be a problem if the state of this append-only table is shared by other operators? For example, doing a delta-join on the append-only table with TTL. Note that the impl of the Lookup executor also contains a cache. 😕

@st1page
Copy link
Contributor Author

st1page commented Feb 2, 2024

Do we allow TTL on index of append only table?

I think it should be the same with the base append only table?

@neverchanje neverchanje mentioned this issue Feb 4, 2024
Closed
@st1page st1page mentioned this issue Feb 26, 2024
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@st1page st1page

Labels
None yet
Projects
None yet
Milestone
release-1.7
3 participants
@hzxa21 @BugenZhao @st1page