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

[CBRD-24581] Assert occurs when using indexes using IFNULL(), NULLIF() functions #4005

Merged
merged 1 commit into from
Dec 20, 2022
Merged

[CBRD-24581] Assert occurs when using indexes using IFNULL(), NULLIF() functions #4005

merged 1 commit into from
Dec 20, 2022

Conversation

ctshim
Copy link
Contributor

@ctshim ctshim commented Dec 16, 2022 •
edited
Loading

http://jira.cubrid.org/browse/CBRD-24581

  • When the return type of the function can vary, the decision of the precision is delayed.
    TP_FLOATING_PRECISION_VALUE is -1, so negative value check in this case is meaningless.
    Remove the assert check and treat it as normal.

@ctshim ctshim self-assigned this Dec 16, 2022
@ctshim ctshim changed the title Assert occurs when using indexes using IFNULL(), NULLIF() functions [CBRD-24581] Assert occurs when using indexes using IFNULL(), NULLIF() functions Dec 16, 2022
@ctshim ctshim merged commit 5c35efd into CUBRID:develop Dec 20, 2022
@ctshim ctshim deleted the CBRD-24581_assert_ifnull_nullif_nvl branch December 20, 2022 23:49
ctshim added a commit to ctshim/cubrid that referenced this pull request Jan 16, 2023
@ctshim
[CBRD-24581] Assert occurs when using indexes using IFNULL(), NULLIF(…
4b3f2d4 
…) functions (CUBRID#4005)

http://jira.cubrid.org/browse/CBRD-24581

* [CBRD-24581] Assert occurs when using indexes using IFNULL(), NULLIF() functions
@ctshim ctshim mentioned this pull request Jan 16, 2023
Merged
mhoh3963 pushed a commit that referenced this pull request Jan 17, 2023
@ctshim
[CBRD-24581] Assert occurs when using indexes using IFNULL(), NULLIF(…
737c486 
…) functions (#4005) (#4048)

http://jira.cubrid.org/browse/CBRD-24581

backport #4005
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hgryoo hgryoo hgryoo approved these changes

@beyondykk9 beyondykk9 beyondykk9 approved these changes

@shparkcubrid shparkcubrid shparkcubrid approved these changes

@hyunikn hyunikn hyunikn approved these changes

@youngjinj youngjinj youngjinj approved these changes

Assignees

@ctshim ctshim

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@ctshim @hgryoo @beyondykk9 @shparkcubrid @hyunikn @youngjinj