Skip to content
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

Fixed array patching #365

Merged
merged 2 commits into from
Oct 17, 2024
Merged

Fixed array patching #365

merged 2 commits into from
Oct 17, 2024

Conversation

hirenkp2000
Copy link
Member

No description provided.

@hirenkp2000 hirenkp2000 requested a review from gunplar October 16, 2024 15:39
Copy link

Code Coverage

Overall Project 42.73% 🍏
Files changed 100% 🍏

Module Coverage
here-naksha-app-service 71.14% 🍏
here-naksha-lib-core 32.85% 🍏
Files
Module File Coverage
here-naksha-app-service WriteFeatureApiTask.java 77.28% 🍏
here-naksha-lib-core PatcherUtils.java 92.86% 🍏

@hirenkp2000 hirenkp2000 marked this pull request as ready for review October 16, 2024 15:44
Copy link

Code Coverage

Overall Project 42.73% 🍏
Files changed 100% 🍏

Module Coverage
here-naksha-app-service 71.14% 🍏
here-naksha-lib-core 32.85% 🍏
Files
Module File Coverage
here-naksha-app-service WriteFeatureApiTask.java 77.28% 🍏
here-naksha-lib-core PatcherUtils.java 92.86% 🍏

@hirenkp2000 hirenkp2000 merged commit cc8d2b8 into v2 Oct 17, 2024
3 checks passed
@hirenkp2000 hirenkp2000 deleted the v2_array_patch_fix branch October 17, 2024 07:37
hirenkp2000 added a commit that referenced this pull request Oct 17, 2024
* Fixed array patching
* corrected patcher test expectation
hirenkp2000 added a commit that referenced this pull request Oct 17, 2024
* Added logs to troubleshoot thread hanging issue
* CASL-258 unreleased lock fix (#358)
* Fixed array patching (#365) (#367)
* corrected patcher test expectation
* updated version
* updated changelog
---------

Co-authored-by: Pawel Mazurek <52866094+cyberhead-pl@users.noreply.github.com>
rlakde added a commit that referenced this pull request Oct 25, 2024
* Fixed array patching (#365)

* Fixed array patching
* corrected patcher test expectation

* Fix thread stuck issue + addtnl logs (#368)

* Added logs to troubleshoot thread hanging issue
* CASL-258 unreleased lock fix (#358)
* Fixed array patching (#365) (#367)
* corrected patcher test expectation
* updated version
* updated changelog
---------

Co-authored-by: Pawel Mazurek <52866094+cyberhead-pl@users.noreply.github.com>

---------

Co-authored-by: Hiren Patel <80465571+hirenkp2000@users.noreply.github.com>
Co-authored-by: Pawel Mazurek <52866094+cyberhead-pl@users.noreply.github.com>
hirenkp2000 pushed a commit that referenced this pull request Nov 26, 2024
* Initial Changes.
* Fix Unit Tests
* Review Fixes
* Rebase CASL-321 (#371)
* Fixed array patching (#365)
* Fix thread stuck issue + addtnl logs (#368)
* CASL-258 unreleased lock fix (#358)
* updated version
* updated changelog
* Update reusable-build-and-publish.yml
* Update gradle.properties
* Update openapi.yaml

---------

Signed-off-by: lakde <rohit.lakde@here.com>
rlakde added a commit that referenced this pull request Dec 12, 2024
* Initial Changes.
* Fix Unit Tests
* Review Fixes
* Rebase CASL-321 (#371)
* Fixed array patching (#365)
* Fix thread stuck issue + addtnl logs (#368)
* CASL-258 unreleased lock fix (#358)
* updated version
* updated changelog
* Update reusable-build-and-publish.yml
* Update gradle.properties
* Update openapi.yaml

---------

Signed-off-by: lakde <rohit.lakde@here.com>
rlakde added a commit that referenced this pull request Jan 16, 2025
* Initial Changes.

Signed-off-by: lakde <rohit.lakde@here.com>

* Fix Unit Tests

Signed-off-by: lakde <rohit.lakde@here.com>

* Review Fixes

Signed-off-by: lakde <rohit.lakde@here.com>

* Update openapi.yaml

* Update gradle.properties

* Update gradle.properties

* Rebase CASL-321 (#371)

* Fixed array patching (#365)

* Fixed array patching
* corrected patcher test expectation

* Fix thread stuck issue + addtnl logs (#368)

* Added logs to troubleshoot thread hanging issue
* CASL-258 unreleased lock fix (#358)
* Fixed array patching (#365) (#367)
* corrected patcher test expectation
* updated version
* updated changelog
---------

Co-authored-by: Pawel Mazurek <52866094+cyberhead-pl@users.noreply.github.com>

---------

Co-authored-by: Hiren Patel <80465571+hirenkp2000@users.noreply.github.com>
Co-authored-by: Pawel Mazurek <52866094+cyberhead-pl@users.noreply.github.com>

* Update reusable-build-and-publish.yml

* Update gradle.properties

* Update openapi.yaml

* Add Interface IExtensionInit

* Add instanceCache logic.

* Create ValueTuple.java

* loaderCache Changes

* Correct logger messages

* Update IExtensionInit description

* Fix bug

* Correct logger in Hub.

* Update IExtensionInit Description.

* Correct removeExtensionFromCache.

* Review Fixes for ExtensionCache

* Update ExtensionCache.java

* Update NakshaHub.java

* Extension Sub Environment Changes (#330) (#392)

* Initial Changes.
* Fix Unit Tests
* Review Fixes
* Rebase CASL-321 (#371)
* Fixed array patching (#365)
* Fix thread stuck issue + addtnl logs (#368)
* CASL-258 unreleased lock fix (#358)
* updated version
* updated changelog
* Update reusable-build-and-publish.yml
* Update gradle.properties
* Update openapi.yaml

---------

Signed-off-by: lakde <rohit.lakde@here.com>

* Update ExtensionCache.java

* Update NakshaHub.java

* Update gradle.properties

* Update openapi.yaml

* Update NakshaVersion.java

* Update gradle.properties

* Update gradle.properties

* Update NakshaVersion.java

* Update NakshaHub.java

* Support multiple JWT public keys (#395) (#396)

* Updated doc about existing JWT loading
* added support for additional pub key
* updated readme
* Updated version to 2.2.1
* Updated doc for pvt and pub key
* Added constants in NakshaHubConfig constructor
* addressed review comments

Co-authored-by: Hiren Patel <80465571+hirenkp2000@users.noreply.github.com>

* Update ExtensionCache.java

* Update gradle.properties

* Update openapi.yaml

* Update NakshaVersion.java

* Update logger as per review comment.

---------

Signed-off-by: lakde <rohit.lakde@here.com>
Co-authored-by: Hiren Patel <80465571+hirenkp2000@users.noreply.github.com>
Co-authored-by: Pawel Mazurek <52866094+cyberhead-pl@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants