Code

Add Highlight In SQL#96

Code

Merged

forestmvey merged 27 commits intointeg_highlight-in-sqlBit-Quill/opensearch-project-sql:integ_highlight-in-sqlfrom dev-highlight-in-sqlBit-Quill/opensearch-project-sql:dev-highlight-in-sqlCopy head branch name to clipboardAug 1, 2022

+641-11

Commits

Commits on Jul 26, 2022

• Add support for highlight to parser and AstExpressionBuilder

  Show description for 1097d9f
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  1097d9f

• Add unit test for highlight in AstExpressionBuilder

  Show description for 1a453e1
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  1a453e1

• Add unit test for highlight in AstBuilderTest

  Show description for 0d5c87b
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  0d5c87b

• Support highlight as an Unresolved expression.

  Show description for 26d0b7e
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  26d0b7e

• Represent highlight as UnresolvedExpression.

  Show description for 3f10b8b
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  3f10b8b

• Support highlight in Analyzer.

  Show description for 543d0d7
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  543d0d7

• Treat highlight as a proper function in AST

  Show description for f47ffe7
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  f47ffe7

• Add support for highlight in Analyzer

  Show description for 5fdb939
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  5fdb939

• Add a simple IT test for highlight.

  Show description for 5c8db0a
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  5c8db0a

• Register highlight function in the BuiltInFunctionRepository

  Show description for ac9f080
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  ac9f080

• Partial support for highlight in physical plan.

  Show description for b526132
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  b526132

• Add HighlightOperator.

  Show description for 807c475
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  807c475

• Highlight alpha complete.

  Show description for 74b6492
  

  MaxKsyunz

  authored and

  forestmvey

  committed
  74b6492

• Initial implementation to upporting highlight('*')

  Show description for ad7affc
  forestmvey

  committed
  ad7affc

• Add support for multiple highlight calls in select statement.

  Show description for a192a21
  forestmvey

  committed
  a192a21

• Cleaning up code, adding copyright, and adding javadocs

  Show description for 092d054
  forestmvey

  committed
  092d054

• Removed OpenSearchLogicalIndexScan highlightFields and dependencies.

  Show description for 9bdbb86
  forestmvey

  committed
  9bdbb86

• Adding tests and touching up code in prep for PR.

  Show description for 9b463cf
  forestmvey

  committed
  9b463cf

• Fixing checkstyle errors.

  Show description for 2648e0c
  forestmvey

  committed
  2648e0c

• Added HighlightOperatorTest and additional testing.

  Show description for 7290e90
  forestmvey

  committed
  7290e90

Commits on Jul 27, 2022

• Added HighlightExpressionTest

  Show description for 2feb2c4
  forestmvey

  committed
  2feb2c4

Commits on Jul 28, 2022

• Improving test coverage, fixing checkstyle errors, fixing jacoco errors.

  Show description for cd6f911
  forestmvey

  committed
  cd6f911

Commits on Jul 29, 2022

• Added javadocs, minor PR revisions, and fixed jacoco errors by improving coverage for OpenSearchIndexScan

  Show description for 3db8788
  forestmvey

  committed
  3db8788

• Code cleanup, adding parsing failure tests, and adding tests for highlight acceptance as a string literal as well as qualified name.

  Show description for 3ee3491
  forestmvey

  committed
  3ee3491

• Removing HighlightOperator functionality and minor revisions based on PR.

  Show description for e63b7b9
  forestmvey

  committed
  e63b7b9

• Removed unnecessary visitHighlight call in PhysicalPlanNodeVisitor

  Show description for 11d834b
  forestmvey

  committed
  11d834b

Commits on Jul 30, 2022

• Fixing formatting errors

  Show description for 373353b
  forestmvey

  committed
  373353b