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

GEODE-4375: Fix problem that an exception occurs when transaction from CacheServer via Pool #1340

Closed

Conversation

masaki-yamakawa
Copy link
Contributor

Thank you for submitting a contribution to Apache Geode.

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

For all changes:

  • Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

  • Has your PR been rebased against the latest commit within the target branch (typically develop)?

  • Is your initial contribution a single, squashed commit?

  • Does gradlew build run cleanly?

  • Have you written or updated unit tests to verify your changes?

  • [N/A] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:

Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to dev@geode.apache.org.

masaki-yamakawa and others added 20 commits January 25, 2018 16:38
…orrectly (apache#1345)

- When overriding a cache listener from a template region this would previously
  add the new listener instead of replacing all existing listeners.
…pache#1341)

* GEODE-4379: Move RemoteOutputStreamServer from gfsh to the manager

- This flips the deployment of jars from a pull (by server) to a push (from
  gfsh).

* GEODE-4379: Update sanctioned-geode-core-serializables.txt for FileUploader$RemoteFile
These calls were trying to set the instance variable to a mock instance.
None of them were actually needed. Removing the method.
…#1324)

* GEODE-4142: save commit message for failover after jta commit.

  check if the jta has completed before retrying the beforeCompeltion and afterCompletion in failover cases.
- This supercedes and deprecates --skip-if-exists
apache#1326)

* GEODE-4094: ClientHealthMonitor may cause an NPE in a ServerConnection

- minor refactoring of AcceptorImpl and Handshake to improve testability
- added a unit test to demonstrate race condition
- refactored connection map into a new object to prevent race
…cket

Inhibit logging of a warning message on IOExceptions if the socket is
closed.

This closes apache#1335
…he#1354)

- This adds a small delay in the async shutdown function so that any client
  responses can be sent. This avoids client getting some form of disconnect
  exception during shutdown.
… enabled for parameterized tests (apache#1318)

- This ensures that if @RunWith(Parameterized.class) is specified for a test class, then
  @Parameterized.UseParametersRunnerFactory(CategoryWithParameterizedRunnerFactory.class)
  is also specified for that class. Without that, the tests are not correctly run as the
  presence of @category breaks the parameterization.
@masaki-yamakawa masaki-yamakawa deleted the feature/GEODE-4375 branch January 30, 2018 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
8 participants