Skip to content

Conversation

rose-m
Copy link
Contributor

@rose-m rose-m commented Feb 2, 2021

Description

Bumps compass-export-to-language to https://github.com/mongodb-js/compass-export-to-language/releases/tag/v6.0.7.

Checklist

  • New tests and/or benchmarks are included
  • Documentation is changed or added

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@rose-m rose-m self-assigned this Feb 2, 2021
@rose-m rose-m force-pushed the COMPASS-4475-force-no-builders-for-java-pipeline branch from 9672ebb to 3d499ea Compare February 3, 2021 08:24
@rose-m rose-m merged commit ae438c6 into master Feb 3, 2021
@rose-m rose-m deleted the COMPASS-4475-force-no-builders-for-java-pipeline branch February 3, 2021 08:27
@daibertdiego
Copy link

Why have you removed this feature? The pipeline in Java using Builders has a much more cleaner code than the version using new Document( ) for everything.

@rose-m
Copy link
Contributor Author

rose-m commented Apr 8, 2021

Hey @daibertdiego - absolutely, we also like the Java Builders fluent API a lot more than the new Document() API. However, since we cannot reliably generate Java code for all of the aggregation stages right now due to a behavioral change (and that would make your code suddenly fail) we had to use this as a temporary fix. We are working on providing the full aggregations stages as fluent API methods and then we'll of course bring this back!

See also for example this ticket in the Java driver: https://jira.mongodb.org/browse/JAVA-3876

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants