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.

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

[CALCITE-2597] RelDataTypeImpl:digest can not describe a type complet… #872

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

huangjiatian
Copy link

…ely.

@danny0405
Copy link
Contributor

danny0405 commented Oct 8, 2018

The digest is overridden by every data type, so what type you mean do not have the field info ?

If the digest is not complete, we can just fix it, a fall back if decision for fieldList is a little confusing.

@vlsi vlsi added returned-with-feedback There are review comments (in JIRA and/or in GitHub) to be implemented before merging the PR discussion-in-jira There's open discussion in JIRA to be resolved before proceeding with the PR invalid labels Jan 8, 2019
@danny0405 danny0405 force-pushed the master branch 2 times, most recently from 80f411d to ca27fe9 Compare November 30, 2019 07:52
@vlsi vlsi force-pushed the master branch 2 times, most recently from 49cb002 to 8768a23 Compare December 29, 2019 12:07
@julianhyde julianhyde force-pushed the main branch 2 times, most recently from 8a5cf83 to cf7f71b Compare June 8, 2023 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion-in-jira There's open discussion in JIRA to be resolved before proceeding with the PR invalid returned-with-feedback There are review comments (in JIRA and/or in GitHub) to be implemented before merging the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants