Skip to content

[FLINK-3587] Bump Calcite version to 1.7.0#1897

Closed
fhueske wants to merge 1 commit intoapache:masterfrom
fhueske:calcite1.7
Closed

[FLINK-3587] Bump Calcite version to 1.7.0#1897
fhueske wants to merge 1 commit intoapache:masterfrom
fhueske:calcite1.7

Conversation

@fhueske
Copy link
Contributor

@fhueske fhueske commented Apr 15, 2016

  • General
    • The pull request references the related JIRA issue
    • The pull request addresses only one issue
    • Each commit in the PR has a meaningful commit message
  • Documentation
    • Documentation has been added for new functionality
    • Old documentation affected by the pull request has been updated
    • JavaDoc for public methods has been added
  • Tests & Build
    • Functionality added by the pull request is covered by tests
    • mvn clean verify has been executed successfully locally or a Travis build has passed

This pull request updates the Apache Calcite dependency to 1.7. With Calcite 1.7 a few method APIs and the some behavior changed.

  • Fix changed API for optimizer cost and cardinality estimates.
  • Add DataSetValues and DataStreamValues due to changed Calcite RelNode generation.
  • Pass TableEnvironment instead of TableConfig for DataSet and DataStream translation.
  • Add methods to create new DataSources to BatchTableEnvironment and StreamTableEnvironment.
  • Remove copied Calcite rule that got fixed.

- Add DataSetValues and DataStreamValues due to changed Calcite RelNode generation.
- Pass TableEnvironment instead of TableConfig for DataSet and DataStream translation.
- Add methods to create new DataSources to BatchTableEnvironment and StreamTableEnvironment.
- Remove copied Calcite rule that got fixed.
@vasia
Copy link
Contributor

vasia commented Apr 18, 2016

Thank you @fhueske. Changes look good to me. +1 to merge.

@zentol
Copy link
Contributor

zentol commented Apr 18, 2016

Merging.

@asfgit asfgit closed this in 367687d Apr 18, 2016
@fhueske fhueske deleted the calcite1.7 branch April 18, 2016 15:13
kl0u pushed a commit to kl0u/flink that referenced this pull request Apr 29, 2016
- Add DataSetValues and DataStreamValues due to changed Calcite RelNode generation.
- Pass TableEnvironment instead of TableConfig for DataSet and DataStream translation.
- Add methods to create new DataSources to BatchTableEnvironment and StreamTableEnvironment.
- Remove copied Calcite rule that got fixed.

This closes apache#1897
StefanRRichter pushed a commit to StefanRRichter/flink that referenced this pull request May 3, 2016
- Add DataSetValues and DataStreamValues due to changed Calcite RelNode generation.
- Pass TableEnvironment instead of TableConfig for DataSet and DataStream translation.
- Add methods to create new DataSources to BatchTableEnvironment and StreamTableEnvironment.
- Remove copied Calcite rule that got fixed.

This closes apache#1897
hequn8128 pushed a commit to hequn8128/flink that referenced this pull request Jun 22, 2017
- Add DataSetValues and DataStreamValues due to changed Calcite RelNode generation.
- Pass TableEnvironment instead of TableConfig for DataSet and DataStream translation.
- Add methods to create new DataSources to BatchTableEnvironment and StreamTableEnvironment.
- Remove copied Calcite rule that got fixed.

This closes apache#1897
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants