forked from apache/calcite-avatica
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Master 2 #4
Merged
Merged
Master 2 #4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ime (Vladimir Ozerov) Close apache#134
…h, Alessandro Solimando)
…va types (Ralph Gasser) Close apache#105
…olimando) 1. Add missing unit tests for CALCITE-3163 2. Reduce warnings in ArrayTypeTest.java 3. Refactor related code to improve readability Close apache#139
…ntent Assertions regarding: * verification of the number of created accessors; * verification of the metadata of the accessor's result set; are removed since they are part of the test setup.
…QL types (Alessandro Solimando) 1. RECORD and RECORD_PROJECTION are now handled in the same way (both require fields) so there is no point keeping both. 2. Adapt LocalService#toResponse after dropping RECORD_PROJECTION to prevent broken clients, and add notes about the change of CursorFactory. 3. Update history.md with details on the breaking change. 4. Add javadoc for deduce method. Close apache#138
Remove html5shiv.min.js/respond.min.js The scripts ensure compatibility with old web browsers such as IE9. However, most (if not all) of these browsers are not used nowadays cause they are very old thus there is no need to use these scripts.
Refactor the default client implementation to use global HTTP Connection Pools instead of one per JDBC connection.
Updating jetty.version causes several SPENGO tests to fail due to a breaking change in Jetty. The threads created by Jetty no longer run as the Subject Jetty was started as. 1. Add new custom ThreadFactrory to obtain the old behavior and overcome failures. 2. Copy most of org.apache.calcite.util.Sources to Avatica, and use them in tests. 3. Remove illegal leading slash in the file paths to avoid test failures on Windows. Close apache#147
This step was missed when the Avatica 1.18.0 was released. Close apache#151
BACK_TICK_BACKSLASH is added to support Google BigQuery's identifier syntax. We also add SINGLE_QUOTE and SINGLE_QUOTE_BACKSLASH, which are not used by any known dialect, for possible future use. Close apache#153
…r later checker-qual has MIT license, so it requires to include the license text when redistributing checker-qual. checker-qual before 3.0.0 did not include the license file as a part of the jar, so we enforce the newer version.
The default Guava version is now 31.0.1-jre, and we allow any Guava version between 14.0.1 and 31.0.1-jre. For versions 22 to 29 inclusive, Guava does not correctly include some MIT licenses for some dependent libraries, so we disable certain Gradle tasks. See .travis.yml and [CALCITE-4815] for details. In the Travis test matrix, add tests for various Guava versions and time zones.
…ATE, TIME or TIMESTAMP elements Support DATE/TIME/TIMESTAMP slot object's content for Date/Time/Timestamp array accessors. Rename method DateFromNumberAccessor#getDate to DateFromNumberAccessor#getNumber. Fix tests that were introduced in the following changes (Alessandro Solimando): * [CALCITE-4757] Allow columns of type Null in ResultSet * [CALCITE-4536] Add support for BIT data type Close apache#154
…d INTEGER and DECIMAL elements Close apache#143
…e.protobuf.UnsafeUtil warnings Close apache#146
…in case of java 8
…o promote release
…xception: Read timed out" after 3 minutes make socket timeout configurable via the new 'http_response_timeout' URL option
- Add JIRA/GitHub links in Git in IDEA - Enable commit inspection in IDEA
…ults Signed-off-by: Mihai Budiu <[email protected]>
Signed-off-by: Mihai Budiu <[email protected]>
Bumps [rexml](https://github.com/ruby/rexml) from 3.2.5 to 3.2.8. - [Release notes](https://github.com/ruby/rexml/releases) - [Changelog](https://github.com/ruby/rexml/blob/master/NEWS.md) - [Commits](ruby/rexml@v3.2.5...v3.2.8) --- updated-dependencies: - dependency-name: rexml dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
Bump dependencies forbiddenapis from 3.4to 3.7 gha actions/checkout from v3 to v4 gha actions/setup-java from v2 to v4 Replaced deprecated `JavaPluginConvention` and `buildDir` in `build.gradle.kts` files
… master_2 For Calcite upgrade
vijayjogi-dm
commented
Jun 14, 2024
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.
ok with current changes
vijayjogi-dm
commented
Jun 14, 2024
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.
ok with apache changes
anand-dm
approved these changes
Jun 14, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.