[VOTE RESULT]
This vote is closed with three (+1) binding votes, and no other votes.
On Fri, Oct 11, 2024 at 6:12 PM Lee Rhodes wrote:
> +1 (binding)
>
> I checked:
> - mvn clean test with 17 ok, fails with 21
> - Notice and License good
>
>
> On Fri, Oct 11, 2024 at 8:26 AM Jon Malkin wrote:
+1 (binding)
I checked:
- mvn clean test with 17 ok, fails with 21
- Notice and License good
On Fri, Oct 11, 2024 at 8:26 AM Jon Malkin wrote:
> +1 (binding)
>
> I checked:
> * LICENSE and NOTICE
> * signatures and checksums for all files in dist
> * mvn clean package works with java 17, fails
AlexanderSaydakov merged PR #69:
URL: https://github.com/apache/datasketches-bigquery/pull/69
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubs
jmalkin commented on PR #69:
URL:
https://github.com/apache/datasketches-bigquery/pull/69#issuecomment-2408228341
I think the last `finally` block in `serialize()` is probably extraneous now
but it shouldn't be harmful.
--
This is an automated message from the Apache Git Service.
To resp
jmalkin commented on code in PR #69:
URL:
https://github.com/apache/datasketches-bigquery/pull/69#discussion_r1797479544
##
theta/sqlx/theta_sketch_agg_int64_lgk_seed_p.sqlx:
##
@@ -89,18 +81,19 @@ export function serialize(state) {
}
} else if (state.sketch != null
Hello All,
This is the core Java component of the DataSketches library that
includes all the sketch algorithms in production-ready packages. These
sketches can be called directly from this component or used in conjunction
with the adaptor components such as Hadoop Pig, Hadoop Hive, or the
aggregato
leerho merged PR #208:
URL: https://github.com/apache/datasketches-website/pull/208
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@data
leerho opened a new pull request, #208:
URL: https://github.com/apache/datasketches-website/pull/208
(no comment)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubsc
AlexanderSaydakov opened a new pull request, #69:
URL: https://github.com/apache/datasketches-bigquery/pull/69
I think this is slightly better and less error-prone. In particular, the
parameter "p" was not passed through serialize in sketch building functions,
which is fixed now.
--
This
leerho merged PR #207:
URL: https://github.com/apache/datasketches-website/pull/207
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@data
leerho opened a new pull request, #207:
URL: https://github.com/apache/datasketches-website/pull/207
Updates Downloads table for Java 6.1.1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the spe
leerho merged PR #206:
URL: https://github.com/apache/datasketches-website/pull/206
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@data
leerho commented on code in PR #206:
URL:
https://github.com/apache/datasketches-website/pull/206#discussion_r1797249451
##
docs/Community/ReleaseProcessForJavaComponents.md:
##
@@ -170,6 +170,7 @@ Also specifies line separator characters for text files.
### Check Primary Zi
[VOTE RESULT]
This vote is now closed as PASSED with 3 (+1 binding) votes, one (+1)
non-binding vote and no other votes.
On Fri, Oct 11, 2024 at 10:13 AM Lee Rhodes wrote:
> +1 (binding)
> Checked
> - mvn clean test for both java8 and 11
> - NOTICE and LICENSE are OK
> - checksums and signature
+1 (binding)
Checked
- mvn clean test for both java8 and 11
- NOTICE and LICENSE are OK
- checksums and signatures OK
On Thu, Oct 10, 2024 at 8:56 AM Jon Malkin wrote:
> +1 (binding)
>
> Checked:
> * signatures and checksums for all release files
> * NOTICE and LICENSE seem unchanged from befor
+1 (binding)
I checked:
* LICENSE and NOTICE
* signatures and checksums for all files in dist
* mvn clean package works with java 17, fails with 8, 11, and 21, as
expected
jon
On Tue, Oct 8, 2024 at 6:30 PM Lee Rhodes wrote:
> *Hello Apache DataSketches PMC and Community,*
>
> 1. This is a c
16 matches
Mail list logo