-
Notifications
You must be signed in to change notification settings - Fork 181
Add License Headers to Workbench #10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…nitial migration pr
joshuali925
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
| describe('Test table display', () => { | ||
| beforeEach(() => { | ||
| cy.visit('app/opendistro-query-workbench'); | ||
| cy.visit('app/opensearch-query-workbench'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this an "API" change, aka did we need to preserve the old path with a redirect?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No not an API change, this changes the url display in OpenSearch-Dashboards. e.g when I visit query workbench, the url will look like localhost:5601/app/opensearch-query-workbench instead of opendistro
…-AOS-231-part-2 Update Vendor Name as of AOS-231
* [Spotless] Applying Google Code Format for integ-tests #10 (#329) * Add spotless apply 33 files. Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * Addressed PR comments. Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * Address PR comments. Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * fixing integ test failure. Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> --------- Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> * Update integ-test/src/test/java/org/opensearch/sql/correctness/tests/TestConfigTest.java Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com> Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> * Apply suggestions from code review Co-authored-by: Guian Gumpac <guian.gumpac@improving.com> Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> * Apply suggestions from code review Co-authored-by: Guian Gumpac <guian.gumpac@improving.com> Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> --------- Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com> Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com> Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com> Co-authored-by: Guian Gumpac <guian.gumpac@improving.com>
* Added support for Timestamp fields in filter Signed-off-by: Vinay Krishna Pudyodu <vinkrish.neo@gmail.com> * Updated the condition in TypeConverted to handle timestamp udt Signed-off-by: Vinay Krishna Pudyodu <vinkrish.neo@gmail.com> * Modifed the updateTimeStampFunction to handle recursively and updated checker method for timestamp udt Signed-off-by: Vinay Krishna Pudyodu <vinkrish.neo@gmail.com> --------- Signed-off-by: Vinay Krishna Pudyodu <vinkrish.neo@gmail.com>
Description
[Describe what this change achieves]
opensearch-query-workbenchIssues Resolved
[List any issues this PR will resolve]
opensearch-query-workbenchCheck List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.