Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Backporting latest change from develop to opendistro-1.11 #945

Conversation

penghuo
Copy link
Contributor

@penghuo penghuo commented Dec 17, 2020

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

harold-wang and others added 6 commits December 15, 2020 16:23
…rch#931)

* Enable count(Date)
Add IT
Add Comparsion Test

* Enable count(Date)
Add IT

* Add comparsion test file 916.txt

* Consolidate count function to accept all field type
…asticsearch#905)

* added metrics in sql new engine query action when errors occur during query execution

* addressed comments

* update

* take all errors from new query engine as server errors
* add right function

* update doc

* address PR comments

* fix build failure

Co-authored-by: Rupal Mahajan <>
* Rename getName to getNameOrAlias

* Use original name as name in JDBC format

* Support alias in CLI

* Use local CLI for doctest

* Add UT

* Fix IT

* Fix IT

* Fix UT

* Update javadoc
…search#942)

* Remove the unsupported type from symbol table

* update doc
@codecov
Copy link

codecov bot commented Dec 17, 2020

Codecov Report

Merging #945 (e80d96b) into opendistro-1.11-rc3 (cc1e595) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                  Coverage Diff                   @@
##             opendistro-1.11-rc3     #945   +/-   ##
======================================================
  Coverage                  99.86%   99.86%           
- Complexity                  2320     2340   +20     
======================================================
  Files                        230      232    +2     
  Lines                       5337     5384   +47     
  Branches                     346      350    +4     
======================================================
+ Hits                        5330     5377   +47     
  Misses                         5        5           
  Partials                       2        2           
Impacted Files Coverage Δ Complexity Δ
...ical/ElasticsearchLogicalPlanOptimizerFactory.java 100.00% <ø> (ø) 1.00 <0.00> (ø)
...ndistroforelasticsearch/sql/analysis/Analyzer.java 100.00% <100.00%> (ø) 44.00 <0.00> (ø)
...rch/sql/analysis/ExpressionReferenceOptimizer.java 100.00% <100.00%> (ø) 12.00 <0.00> (ø)
...opendistroforelasticsearch/sql/expression/DSL.java 100.00% <100.00%> (ø) 129.00 <1.00> (+1.00)
...icsearch/sql/expression/ExpressionNodeVisitor.java 100.00% <100.00%> (ø) 14.00 <1.00> (ø)
...relasticsearch/sql/expression/NamedExpression.java 100.00% <100.00%> (ø) 6.00 <1.00> (ø)
...sql/expression/aggregation/AggregatorFunction.java 100.00% <100.00%> (ø) 31.00 <2.00> (-6.00)
...h/sql/expression/function/BuiltinFunctionName.java 100.00% <100.00%> (ø) 3.00 <0.00> (ø)
...lasticsearch/sql/expression/text/TextFunction.java 100.00% <100.00%> (ø) 34.00 <2.00> (+2.00)
...arch/sql/planner/physical/AggregationOperator.java 100.00% <100.00%> (ø) 7.00 <0.00> (ø)
... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc1e595...e80d96b. Read the comment docs.

@penghuo penghuo changed the title Merge latest change from develop Backporting latest change from develop to opendistro-1.11 Dec 17, 2020
@penghuo penghuo merged commit db98383 into opendistro-for-elasticsearch:opendistro-1.11-rc3 Dec 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants