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

Bump extra-enforcer-rules from 1.5.1 to 1.6.1 #1076

Conversation

dependabot[bot]
Copy link

@dependabot dependabot bot commented on behalf of github Jan 19, 2023

Bumps extra-enforcer-rules from 1.5.1 to 1.6.1.

Release notes

Sourced from extra-enforcer-rules's releases.

1.6.1

🐛 Bug Fixes

📦 Dependency updates

👻 Maintenance

1.6.0

🚀 New features and improvements

📦 Dependency updates

... (truncated)

Commits
  • 45d56eb [maven-release-plugin] prepare release extra-enforcer-rules-1.6.1
  • 4f6a54c Execute m-invoker-p in run-its profile
  • aaf4120 Bump maven-common-artifact-filters from 3.3.0 to 3.3.1
  • be34f37 Use all remote repositories for artifacts resolving
  • 2ee33d0 [maven-release-plugin] prepare for next development iteration
  • f89ea21 [maven-release-plugin] prepare release extra-enforcer-rules-1.6.0
  • 5d10f5c Bump mojo-parent from 68 to 69
  • 8f7eb0f Use shard GitHub actions from Apache
  • d342299 ignore Maven Core updates
  • 3195d55 Remove maven-compat from project dependencies
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [extra-enforcer-rules](https://github.com/mojohaus/extra-enforcer-rules) from 1.5.1 to 1.6.1.
- [Release notes](https://github.com/mojohaus/extra-enforcer-rules/releases)
- [Commits](mojohaus/extra-enforcer-rules@extra-enforcer-rules-1.5.1...extra-enforcer-rules-1.6.1)

---
updated-dependencies:
- dependency-name: org.codehaus.mojo:extra-enforcer-rules
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Jan 19, 2023
@dependabot @github
Copy link
Author

dependabot bot commented on behalf of github Jan 19, 2023

The following labels could not be found: Source: Internal.

@dependabot @github
Copy link
Author

dependabot bot commented on behalf of github Mar 30, 2023

Superseded by #1225.

@dependabot dependabot bot closed this Mar 30, 2023
@dependabot dependabot bot deleted the dependabot/maven/org.codehaus.mojo-extra-enforcer-rules-1.6.1 branch March 30, 2023 03:06
TomaszGaweda pushed a commit that referenced this pull request May 14, 2024
…572] (#1076)

`AddIndexBackupOperation` should be allowed in passive state for the
same reason why `AddIndexOperation` is.

The issue became visible after improving assertions in
https://github.com/hazelcast/hazelcast-mono/pull/1009, before it just
failed (with warnings in logs) during proxy initialisation. This is a
missing part of https://github.com/hazelcast/hazelcast-mono/pull/596.

This PR fixes useless warnings `java.lang.IllegalStateException: Cluster
is in PASSIVE state! Operation:
com.hazelcast.map.impl.operation.AddIndexBackupOperation`. Map proxy
initialization on primary does not fail - backup operation errors are
ignored, so indexes should be fine. The only cases where it might matter
are:

1. for nodes that do not own any partitions but that is very unlikely
(see comment in `AddIndexBackupOperation.runInternal`).
2. if the members are restarted quickly one after another. Backup
operation has 5s timeout and in case of backup operation error no
response is returned. This may cause MapProxy not being initialized yet
when another member joins, so it will not be sent in
`PostJoinProxyOperation`.

There is a discrepancy in handling `Backup` which contains
not-`AllowedDuringPassiveState` operation in PASSIVE state between
normal execution (allowed! - does not check nested operation) and
offloaded execution (fails with `IllegalStateException` - steps engine
checks preconditions for nested operation). That explains the difference
between force-offload and normal behavior. This might be fixed
separately as it is error prone, but generally our operations should be
reasonably defined and not cause problems.

Fixes HZ-4582, HZ-4572
Fixes https://github.com/hazelcast/hazelcast-enterprise/issues/7076
Fixes https://github.com/hazelcast/hazelcast-enterprise/issues/7075

GitOrigin-RevId: ef1eec30f3a78f07c0c1e38ae6fbffb482a6a9f4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants