Skip to content
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

Pull most recent changes from the upstream project #26

Merged
merged 9 commits into from
Jan 5, 2022

Conversation

Yury-Fridlyand
Copy link

Description

Issues Resolved

PR #355 failed because of misconfiguration in the upstream project. It was offered to take the most recent changes and try re-applying changes from opensearch-project#355.

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

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.

joshuali925 and others added 9 commits December 15, 2021 11:11
* Bump log4j to 2.16 and plugin to 1.2.2

Signed-off-by: Joshua Li <[email protected]>

* Add release notes

Signed-off-by: Joshua Li <[email protected]>

* Bump bwc to 1.2.2-SNAPSHOT

Signed-off-by: Joshua Li <[email protected]>

* Use opensearch_version for bwc

Signed-off-by: Joshua Li <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
added untriaged and nolabel badges
* Added Tableau Connector to OpenSearch SQL

Signed-off-by: Guian Gumpac <[email protected]>

* Added CEILING and FLOOR functions

Signed-off-by: Guian Gumpac <[email protected]>

* Added IFNULL function

Signed-off-by: Guian Gumpac <[email protected]>

* Added Kyle's fix for ADDDATE and SUBDATE with resulting 00:00:00 being unexpectedly null

Signed-off-by: Guian Gumpac <[email protected]>

* Added MIN and MAX for two string arguments

Signed-off-by: Guian Gumpac <[email protected]>

* Fixed return type for MID function redefenitions, part of AOS-202

Signed-off-by: Yury Fridlyand <[email protected]>

* Added CAST functions to convert to int or string as part of AOS-202

Signed-off-by: Yury Fridlyand <[email protected]>

* Added HEXBINX and HEXBINY functions from the templace without any modifications

Signed-off-by: Yury Fridlyand <[email protected]>

* Fixed cast to use the OSSQL type

Signed-off-by: Yury Fridlyand <[email protected]>

* Added parenthesis to avoid /issues/293

Signed-off-by: Yury Fridlyand <[email protected]>

* Modified Timestamp conversion formula

Signed-off-by: Guian Gumpac <[email protected]>

* Added comment to TimestampType

Signed-off-by: Guian Gumpac <[email protected]>

* Updated Company Name accroding to the customer's request

Signed-off-by: Yury Fridlyand <[email protected]>

* Updated Vendor Name as well

Signed-off-by: Yury Fridlyand <[email protected]>

* Added few small changes to simplify connector developing and debugging

Signed-off-by: Yury Fridlyand <[email protected]>

* Typo fix

Signed-off-by: Yury Fridlyand <[email protected]>

* Added a simple test from TDVT: case bool0 from test calcs_data (an expression test)
Some mofidication were made in the test framework to make the output more readable

Signed-off-by: Yury Fridlyand <[email protected]>

* Modified the driver to recognize its support for conversion of types

Signed-off-by: Guian Gumpac <[email protected]>

* Removed changes from a different branch

Signed-off-by: Guian Gumpac <[email protected]>

* Removed changes from a different branch

Signed-off-by: Guian Gumpac <[email protected]>

* Made the connector convert keyword columns to string columns

Signed-off-by: Guian Gumpac <[email protected]>

* Fix requested by PR review

Signed-off-by: Yury Fridlyand <[email protected]>

* Modified integration tests to reflect support for conversion types

Signed-off-by: Guian Gumpac <[email protected]>

* Removed connector changes

Signed-off-by: Guian Gumpac <[email protected]>

* Change values to bitmasks

Signed-off-by: Guian Gumpac <[email protected]>

* Add timestamp as a type for the driver

Signed-off-by: Guian Gumpac <[email protected]>

* Added timestamp to SQLGetTypeInfo for the ODBC Driver

Signed-off-by: Guian Gumpac <[email protected]>

* Reverted some previous changes

Signed-off-by: Guian Gumpac <[email protected]>

* Removed generated test output file

Signed-off-by: Guian Gumpac <[email protected]>

Co-authored-by: Yury Fridlyand <[email protected]>
Co-authored-by: Yury-Fridlyand <[email protected]>
@Yury-Fridlyand Yury-Fridlyand requested review from kylepbit, a user and raymond-lum January 4, 2022 21:15
@codecov
Copy link

codecov bot commented Jan 4, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@1dfaa1b). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #26   +/-   ##
=======================================
  Coverage        ?   99.90%           
  Complexity      ?     2691           
=======================================
  Files           ?      256           
  Lines           ?     6533           
  Branches        ?      422           
=======================================
  Hits            ?     6527           
  Misses          ?        5           
  Partials        ?        1           
Flag Coverage Δ
sql-engine 99.90% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out 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 1dfaa1b...7af51d3. Read the comment docs.

@Yury-Fridlyand Yury-Fridlyand merged commit b10646c into Bit-Quill:main Jan 5, 2022
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.

5 participants