gh-2662: Investigate replacing java.util.logging.Logger with SLF4J #2663
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
Codecov Report
@@ Coverage Diff @@
## develop #2663 +/- ##
=============================================
- Coverage 66.71% 66.22% -0.50%
Complexity 2521 2521
=============================================
Files 997 962 -35
Lines 32492 31010 -1482
Branches 3936 3729 -207
=============================================
- Hits 21678 20535 -1143
+ Misses 9117 8851 -266
+ Partials 1697 1624 -73
Continue to review full report at Codecov.
|
GCHQDeveloper314
approved these changes
Jun 6, 2022
t92549
approved these changes
Jun 6, 2022
t92549
pushed a commit
that referenced
this pull request
Jun 6, 2022
t92549
added a commit
that referenced
this pull request
Jun 7, 2022
t92549
added a commit
that referenced
this pull request
May 12, 2023
* Update docs links in README * Revert "Update docs links in README" This reverts commit 2d87bf3. * gh-2559 added hasTrait, its handler and corresponding tests (#2561) * gh-2559 added hasTrait, its handler and corresponding tests * gh-2559 amended comment in HasTrait, added additional tests for the handler * gh-2559 null check & tidy test Co-authored-by: t92549 <80890692+t92549@users.noreply.github.com> * Update links to Gaffer docs in all files (#2566) Co-authored-by: t92549 <80890692+t92549@users.noreply.github.com> * gh-2604: Deprecates Parquet (#2606) * gh-2556: Add HBaseStore deprecated tag, javadocs and note to README (#2605) * gh-2556: Add HBaseStore deprecated tag, javadocs and note to README * gh-2556: Amend version number * Upgrade Hazelcast version to 5.1.1 (#2612) Co-authored-by: t92549 <80890692+t92549@users.noreply.github.com> * Gh-2602: Replace Log4j with Reload4j (#2609) * Replace Log4j with Reload4j Also add enforcer plugin to prevent future problems * FederatedStoreGetTraitsTest did not delete the cache service before carrying out each test causing it to fail when run as part of the maven tests. In isolation it passed. Found that the FederatedGraphStorageTraitsTest was also not clearing the cache. Added the before and after cache clearance and bumped the JUnit version of FederatedStoreGetTraitsTest from 4 to 5. Tests now pass (#2617) Co-authored-by: r32575 <63105662+r32575@users.noreply.github.com> * Gh 2562 GetElementsWithinSet not allowing for the exclusion of properties - Gaffer v1 (#2597) * Fixed bug gh-2562. View properties are now being filtered. Added tests * After a review moved the return statements inside the doPostFilter if-check and changed a conditional fail to an assertThat test * Removed the try-finally blocks as the classes implement CloseableIterable * Added failure message to test * Adjusted javadoc formatting * Removed rogue letter Co-authored-by: t92549 <80890692+t92549@users.noreply.github.com> * prepare release gaffer2-1.22.0 * prepare for next development iteration * gh-2618: Cleanup inconsistent use of logging strings (#2651) * Found 1 extra inconsistency than the mentioned one * found and fixed other inconsitencies * further fixes * further fixes 2 * fixed 3 out of 4 checkstyle errors - 1 more to fix * all checkstyle errors fixed * removed use of String.format * removed another instance of using String.format * more changes * additional change * reverted changes * replaced to SLF4J (#2663) * gh-2669: Added new aggregation function endpoints (#2670) * gh-2673: Fix links to koryphe core (#2674) * Gh 2705 operation details spring (#2706) * gh-2705: Cherry pick new spring endpoint * gh-2696: Added tests for new all operation details endpoint (#2700) * gh-2696: Added tests for new all operation details endpoint * gh-2696: Spotless apply * gh-2705: Copyright fix * Gh-2745: Fix crown copyright link Fixed Crown Copyright Link on Line 44 * Gh-2776: Create issue templates (#2777) * Create issue templates * Simplify bug report * build(deps): bump hazelcast (#2848) Bumps [hazelcast](https://github.com/hazelcast/hazelcast) from 5.1.1 to 5.1.3. - [Release notes](https://github.com/hazelcast/hazelcast/releases) - [Commits](hazelcast/hazelcast@v5.1.1...v5.1.3) --- updated-dependencies: - dependency-name: com.hazelcast:hazelcast dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * GH-2845: updt class var parameter to use Map interface (#2846) Co-authored-by: t92549 <80890692+t92549@users.noreply.github.com> * Gh-2914: Remove redundant constants class (#2927) * #2914 Issue: Remove redundant constants class * Simplify charset * Fix import * Remove erroneous use of charset with JSONSerialiser.serialise * Use StandardCharsets inbuilt function --------- Co-authored-by: GCHQDeveloper314 <94527357+GCHQDeveloper314@users.noreply.github.com> * prepare release gaffer2-1.23.0 * prepare for next development iteration * Fix field names in ToCsv * Refactor to string enums --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: GCHQDeveloper314 <94527357+GCHQDeveloper314@users.noreply.github.com> Co-authored-by: p3430233 <61738524+p3430233@users.noreply.github.com> Co-authored-by: r32575 <63105662+r32575@users.noreply.github.com> Co-authored-by: Gaffer <github-actions@github.com> Co-authored-by: Robert <33911814+RobertG111@users.noreply.github.com> Co-authored-by: Anirban Patra <96457415+AnirbanPatragithub@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: kvaithin <68995267+kvaithin@users.noreply.github.com> Co-authored-by: Brijesh Yadav <brijeshyadavjthj@gmail.com>
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.
Hi. I have replaced instances of java.util.logging.Logger to SLF4J. I have also cleaned up inconsistent use of logging strings in the files that were using java.uitl.logging.Logger. Let me know if any further changes are required. Many thanks.