You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datasketches.apache.org by le...@apache.org on 2022/12/14 17:12:25 UTC
[datasketches-java] branch master updated (c29f4c1b -> 2f8c8ca2)
This is an automated email from the ASF dual-hosted git repository.
leerho pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/datasketches-java.git
from c29f4c1b Merge pull request #421 from p-/p-/add-codeql-workflow
add 12d1874c This fixes all of the "High", "Error", and "Warnings" issues detected by CodeQL. However, there were some that I classified as "Won't Fix" and "False Positive". But I'm not sure if CodeQL will remember those markings.
add c95e30df Had to do some workarounds to prevent CodeQL from creating warnings that block successful tests.
add bb78da43 Two more CodeQL warnings showed up. Fixed.
new 2f8c8ca2 Merge pull request #422 from apache/fix_codeql_alerts
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.github/workflows/codeql.yml | 1 +
.../java/org/apache/datasketches/common/Util.java | 30 +++++++++++++++++++++-
.../java/org/apache/datasketches/cpc/CpcUnion.java | 6 ++++-
.../org/apache/datasketches/cpc/PreambleUtil.java | 2 +-
.../java/org/apache/datasketches/fdt/Group.java | 19 ++++++++++----
.../java/org/apache/datasketches/hll/HllUtil.java | 2 +-
.../sampling/ReservoirItemsSketch.java | 2 +-
.../sampling/ReservoirLongsSketch.java | 2 +-
.../datasketches/tuple/QuickSelectSketch.java | 2 +-
.../org/apache/datasketches/common/UtilTest.java | 18 +++++++++++++
.../datasketches/cpc/CpcCompressionTest.java | 2 +-
.../org/apache/datasketches/cpc/TestAllTest.java | 2 +-
.../apache/datasketches/frequencies/DistTest.java | 4 +--
.../org/apache/datasketches/hll/UnionTest.java | 5 ++--
.../quantiles/CustomQuantilesTest.java | 7 ++---
.../quantiles/DirectUpdateDoublesSketchTest.java | 2 +-
.../quantiles/HeapCompactDoublesSketchTest.java | 2 +-
.../GenericInequalitySearchTest.java | 30 +++++++++++-----------
.../quantilescommon/InequalitySearchTest.java | 2 +-
.../sampling/ReservoirLongsSketchTest.java | 21 +++------------
.../datasketches/theta/BackwardConversions.java | 2 +-
.../theta/ConcurrentHeapQuickSelectSketchTest.java | 30 ++++++++++------------
.../theta/DirectQuickSelectSketchTest.java | 27 +++++++++----------
.../datasketches/theta/HeapAlphaSketchTest.java | 20 +++++++--------
.../theta/HeapQuickSelectSketchTest.java | 20 +++++++--------
25 files changed, 148 insertions(+), 112 deletions(-)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datasketches.apache.org
For additional commands, e-mail: commits-help@datasketches.apache.org
[datasketches-java] 01/01: Merge pull request #422 from apache/fix_codeql_alerts
Posted by le...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
leerho pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/datasketches-java.git
commit 2f8c8ca2f75f25d54adc79ff15b8d638da96bd90
Merge: c29f4c1b bb78da43
Author: Lee Rhodes <le...@users.noreply.github.com>
AuthorDate: Wed Dec 14 09:12:19 2022 -0800
Merge pull request #422 from apache/fix_codeql_alerts
This fixes all of the "High", "Error", and "Warnings" issues detected by
.github/workflows/codeql.yml | 1 +
.../java/org/apache/datasketches/common/Util.java | 30 +++++++++++++++++++++-
.../java/org/apache/datasketches/cpc/CpcUnion.java | 6 ++++-
.../org/apache/datasketches/cpc/PreambleUtil.java | 2 +-
.../java/org/apache/datasketches/fdt/Group.java | 19 ++++++++++----
.../java/org/apache/datasketches/hll/HllUtil.java | 2 +-
.../sampling/ReservoirItemsSketch.java | 2 +-
.../sampling/ReservoirLongsSketch.java | 2 +-
.../datasketches/tuple/QuickSelectSketch.java | 2 +-
.../org/apache/datasketches/common/UtilTest.java | 18 +++++++++++++
.../datasketches/cpc/CpcCompressionTest.java | 2 +-
.../org/apache/datasketches/cpc/TestAllTest.java | 2 +-
.../apache/datasketches/frequencies/DistTest.java | 4 +--
.../org/apache/datasketches/hll/UnionTest.java | 5 ++--
.../quantiles/CustomQuantilesTest.java | 7 ++---
.../quantiles/DirectUpdateDoublesSketchTest.java | 2 +-
.../quantiles/HeapCompactDoublesSketchTest.java | 2 +-
.../GenericInequalitySearchTest.java | 30 +++++++++++-----------
.../quantilescommon/InequalitySearchTest.java | 2 +-
.../sampling/ReservoirLongsSketchTest.java | 21 +++------------
.../datasketches/theta/BackwardConversions.java | 2 +-
.../theta/ConcurrentHeapQuickSelectSketchTest.java | 30 ++++++++++------------
.../theta/DirectQuickSelectSketchTest.java | 27 +++++++++----------
.../datasketches/theta/HeapAlphaSketchTest.java | 20 +++++++--------
.../theta/HeapQuickSelectSketchTest.java | 20 +++++++--------
25 files changed, 148 insertions(+), 112 deletions(-)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datasketches.apache.org
For additional commands, e-mail: commits-help@datasketches.apache.org