-
Notifications
You must be signed in to change notification settings - Fork 51
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
v2.0.2 release #558
v2.0.2 release #558
Conversation
…irements (#510) * try whitesource token * Remove cla workflow * Update README.md with cla and formatting requirements * Update README.md --------- Co-authored-by: Xin Huang <xin.huang@snowflake.com>
…0-jre (#534) fix: pom.xml to reduce vulnerabilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JAVA-COMGOOGLEGUAVA-5710356 Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: Lukas Sembera <lukas.sembera@snowflake.com>
This PR retries SSLException, which we have been seeing in client logs. It additionally evicts idle connections after 60s of inactivity.
…546) * update pom for vulns * update pom * use fasterxml version * remove hadoop exclusion * mvn install passes * passes * autoformatting * remove dependency pom * relocate parquet-hadoop * dont relocate * relocate just airlift not parquet-hadoop
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, cla bot gate failing is ok since we are removing it in this pr
pom.xml
Outdated
@@ -6,7 +6,7 @@ | |||
<!-- Arifact name and version information --> | |||
<groupId>net.snowflake</groupId> | |||
<artifactId>snowflake-ingest-sdk</artifactId> | |||
<version>2.0.1</version> | |||
<version>2.0.2-SNAPSHOT</version> |
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.
Do we need your other PR as well?
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.
Also confused about the 2 PRs.
Should this just be 2.0.2 ?
Snapshot release should be 2.0.3-SNAPSHOT
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.
We're releasing on https://github.com/snowflakedb/snowflake-ingest-java/tree/tzhang-si-ga, so I'm porting every change that needs to be included manually one by one, this change comes from e9103dd, I will cherry-pick another which will override this to 2.0.2, see 79b4ea2
Codecov Report
@@ Coverage Diff @@
## tzhang-si-ga #558 +/- ##
================================================
+ Coverage 78.39% 78.93% +0.53%
================================================
Files 75 75
Lines 4666 4685 +19
Branches 411 413 +2
================================================
+ Hits 3658 3698 +40
+ Misses 827 808 -19
+ Partials 181 179 -2
... and 2 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Upgrade version to v2.0.2, this will be used as our first GA release!
Release v2.0.2, this will be used as our first GA release! This release is on branch https://github.com/snowflakedb/snowflake-ingest-java/tree/tzhang-si-ga since we only want to include selected changes on top of v2.0.1 to make sure it's a stable release, anything feature related will be released in v2.0.3 in the future.