Skip to content

Conversation

@GabeFernandez310
Copy link

@GabeFernandez310 GabeFernandez310 commented Apr 26, 2023

Description

Updates the functions list to include those newly supported for SQL queries.

PDF of Printed Webpage:
Functions - OpenSearch documentation.pdf

Issues Resolved

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: GabeFernandez310 <Gabriel.Fernandez@improving.com>
@GabeFernandez310 GabeFernandez310 requested a review from a team April 26, 2023 15:22
tan | `tan(number T) -> double` | `SELECT tan(0.5) FROM my-index LIMIT 1`

## Date and time
Functions marked with * are not yet implemented in PPL, with plans to support them in the near future.

Choose a reason for hiding this comment

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

Suggested change
Functions marked with * are not yet implemented in PPL, with plans to support them in the near future.
Functions marked with * have future PPL implementation on roadmap.

@Yury-Fridlyand
Copy link

Missing: addtime, convert_tz, curtime and more. Please, compare list with DateTimeFunction.java or SQL plugin docs.

Signed-off-by: GabeFernandez310 <Gabriel.Fernandez@improving.com>
Signed-off-by: GabeFernandez310 <Gabriel.Fernandez@improving.com>
Signed-off-by: GabeFernandez310 <Gabriel.Fernandez@improving.com>
@GabeFernandez310
Copy link
Author

Missing: addtime, convert_tz, curtime and more. Please, compare list with DateTimeFunction.java or SQL plugin docs.

Fixed. Added all functions currently existing in DateTimeFunction.java

@GabeFernandez310 GabeFernandez310 merged commit d8766a6 into integ-date-time-changes Apr 28, 2023
matthewryanwells pushed a commit that referenced this pull request Jun 8, 2023
* Merge pull request #1 from Yury-Fridlyand/dev-update-sql-relevance-docs

Update SQL plugin relevance functions documentation.

Co-authored-by: MaxKsyunz <maxk@bitquilltech.com>
Signed-off-by: Yury Fridlyand <yuryf@bitquilltech.com>

* Address PR feedback.

Signed-off-by: Yury Fridlyand <yuryf@bitquilltech.com>

* Address PR feedback by @joshuali925.

Signed-off-by: Yury Fridlyand <yuryf@bitquilltech.com>

* Remove PPL page from Observability Plugin. Add link to Observability page. Make some simple formatting changes

Signed-off-by: Naarcha-AWS <naarcha@amazon.com>

* Reword paragraph

Signed-off-by: Naarcha-AWS <naarcha@amazon.com>

* Adds SQL and PPL API and other SQL plugin changes

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

* Formatting changes

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

* Incorporates editorial comments

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

Signed-off-by: Yury Fridlyand <yuryf@bitquilltech.com>
Signed-off-by: Naarcha-AWS <naarcha@amazon.com>
Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>
Co-authored-by: Yury Fridlyand <yuryf@bitquilltech.com>
Co-authored-by: MaxKsyunz <maxk@bitquilltech.com>
Co-authored-by: Fanit Kolchina <kolchfa@amazon.com>
matthewryanwells pushed a commit that referenced this pull request Jun 8, 2023
…t#2374)

* for new page how to analyze Jaeger trace data

Signed-off-by: alicejw <alicejw@amazon.com>

* remove old image

Signed-off-by: alicejw <alicejw@amazon.com>

* for new information and doc writer checklist

Signed-off-by: alicejw <alicejw@amazon.com>

* for new information and doc writer checklist

Signed-off-by: alicejw <alicejw@amazon.com>

* small rewrite

Signed-off-by: alicejw <alicejw@amazon.com>

* new clean images from Dashboards URL directly

Signed-off-by: alicejw <alicejw@amazon.com>

* for additional information

Signed-off-by: alicejw <alicejw@amazon.com>

* remove blank lines

Signed-off-by: alicejw <alicejw@amazon.com>

* for tech review feedback updates

Signed-off-by: alicejw <alicejw@amazon.com>

* add requirements section

Signed-off-by: alicejw <alicejw@amazon.com>

* for new procedure

Signed-off-by: alicejw <alicejw@amazon.com>

* for tech review feedback updates

Signed-off-by: alicejw <alicejw@amazon.com>

* continued updates

Signed-off-by: alicejw <alicejw@amazon.com>

* for docker compose file instructions

Signed-off-by: alicejw <alicejw@amazon.com>

* for docker usage instruction

Signed-off-by: alicejw <alicejw@amazon.com>

* for step 2 view dashboards

Signed-off-by: alicejw <alicejw@amazon.com>

* for additional link provided in tech review

Signed-off-by: alicejw <alicejw@amazon.com>

* for link to index page to introduce the feature

Signed-off-by: alicejw <alicejw@amazon.com>

* final checklist

Signed-off-by: alicejw <alicejw@amazon.com>

* add warning not to use sample file in prod env

Signed-off-by: alicejw <alicejw@amazon.com>

* updated docker file that is safe for prod env, remove warning note for previous file

Signed-off-by: alicejw <alicejw@amazon.com>

* for small update to parent page

Signed-off-by: alicejw <alicejw@amazon.com>

* for tech review

Signed-off-by: alicejw <alicejw@amazon.com>

* typo fix for font

Signed-off-by: alicejw <alicejw@amazon.com>

* for doc review #1 feedback updates

Signed-off-by: alicejw <alicejw@amazon.com>

* for doc review feedback #2 updates

Signed-off-by: alicejw <alicejw@amazon.com>

* for a couple minor changes

Signed-off-by: alicejw <alicejw@amazon.com>

* spell out dashboard URI directly to trace analytics for accessibility. With URL obfuscated, this would not get spelled out in the vision-impaired app that scans through.it would simply read the title only

Signed-off-by: alicejw <alicejw@amazon.com>

* need to add additional step from eng to generate sample data

Signed-off-by: alicejw <alicejw@amazon.com>

* for additional step image of sample app

Signed-off-by: alicejw <alicejw@amazon.com>

* rename step numbers

Signed-off-by: alicejw <alicejw@amazon.com>

* minor fix heading levels

Signed-off-by: alicejw <alicejw@amazon.com>

* updates recommended by the editorial reviewer

Signed-off-by: alicejw <alicejw@amazon.com>

* clarify Spans window function

Signed-off-by: alicejw <alicejw@amazon.com>

* clarified individual trace details section

Signed-off-by: alicejw <alicejw@amazon.com>

Signed-off-by: alicejw <alicejw@amazon.com>
MitchellGale pushed a commit that referenced this pull request Mar 27, 2024
* Updates Cohere connector info in connectors.md

Addresses #1 in opensearch-project#6648

Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>

* Added link to github

Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>

* Update _ml-commons-plugin/remote-models/connectors.md

Co-authored-by: Yaliang Wu <ylwu@amazon.com>
Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>

* Fixed link formatting

Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>

* Update _ml-commons-plugin/remote-models/connectors.md

Co-authored-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>

---------

Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>
Co-authored-by: Yaliang Wu <ylwu@amazon.com>
Co-authored-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
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.

6 participants