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

persistent indexed timestamp(posting_date, posting_time) column on stock ledger entry #38364

Closed
ankush opened this issue Nov 27, 2023 · 0 comments · Fixed by #39800
Closed

persistent indexed timestamp(posting_date, posting_time) column on stock ledger entry #38364

ankush opened this issue Nov 27, 2023 · 0 comments · Fixed by #39800
Assignees
@rohitwaghchaure
Labels
performance

Comments

@ankush
Copy link
Member

ankush commented Nov 27, 2023 •
edited
Loading

persistent indexed timestamp(posting_date, posting_time) column seems like a much better solution than these query hacks.

This can be done in two ways:

  1. Add the column in schema and handle it ourselves + index it.
  2. Add generated column and index it.

Both have pros and cons.

cc: @rohitwaghchaure

Originally posted by @ankush in #38295 (comment)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rohitwaghchaure rohitwaghchaure

Labels
performance
Projects
None yet
Milestone
No milestone
2 participants
@rohitwaghchaure @ankush