Skip to content

[CALCITE-7404] Incorrect Field Alias in MongoDB project Stage#4774

Merged
xuzifu666 merged 1 commit intoapache:mainfrom
cjj2010:mongodb-agg-function
Feb 5, 2026
Merged

[CALCITE-7404] Incorrect Field Alias in MongoDB project Stage#4774
xuzifu666 merged 1 commit intoapache:mainfrom
cjj2010:mongodb-agg-function

Conversation

@cjj2010
Copy link

@cjj2010 cjj2010 commented Jan 28, 2026

@cjj2010 cjj2010 changed the title [CALCITE-7404] Mongodb aggfuntion not support condition filter [CALCITE-7404] Mongodb aggfunction not support condition filter Jan 28, 2026
@cjj2010 cjj2010 changed the title [CALCITE-7404] Mongodb aggfunction not support condition filter [CALCITE-7404] Mongo adapter aggfunction not support condition filter Jan 28, 2026
@cjj2010 cjj2010 changed the title [CALCITE-7404] Mongo adapter aggfunction not support condition filter [CALCITE-7404] Incorrect Field Alias in MongoDB $project Stage Jan 29, 2026
@cjj2010 cjj2010 changed the title [CALCITE-7404] Incorrect Field Alias in MongoDB $project Stage [CALCITE-7404] Incorrect Field Alias in MongoDB project Stage Jan 29, 2026
@xuzifu666
Copy link
Member

Jira mentioned some issues that need to be confirmed first.

Copy link
Member

@xuzifu666 xuzifu666 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From a MongoDB syntax perspective, the current modifications should be fine.

@xuzifu666
Copy link
Member

Please fix the conflicts.

@cjj2010 cjj2010 force-pushed the mongodb-agg-function branch from 7f1e534 to c89bcff Compare February 2, 2026 08:32
@cjj2010
Copy link
Author

cjj2010 commented Feb 2, 2026

Please fix the conflicts.

done, thanks

@cjj2010 cjj2010 force-pushed the mongodb-agg-function branch from c89bcff to 4e54b11 Compare February 5, 2026 02:17
@xuzifu666 xuzifu666 added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Feb 5, 2026
@xuzifu666 xuzifu666 merged commit 5543694 into apache:main Feb 5, 2026
33 of 34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTM-will-merge-soon Overall PR looks OK. Only minor things left.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants