You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@geode.apache.org by sboorlagadda <gi...@git.apache.org> on 2015/10/28 19:43:39 UTC

[GitHub] incubator-geode pull request: Fixes GEODE-485.

GitHub user sboorlagadda opened a pull request:

    https://github.com/apache/incubator-geode/pull/26

    Fixes GEODE-485. 

    Logging at debug level on a SocketException while closing the connection to avoid any suspect strings written to logs.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/sboorlagadda/incubator-geode feature/GEODE-409

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-geode/pull/26.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #26
    
----
commit 9957c6416a99004058ab242edc4f3b711d3cfaf0
Author: Dave Barnes <db...@pivotal.io>
Date:   2015-10-21T22:58:48Z

    GEODE-465 changed 2 occurrences of PartionedResolver to PartitionResolver in Javadocs

commit d812f901033ec3819a5d715bb43380648ff83ef4
Author: Mark Bretl <mb...@pivotal.io>
Date:   2015-10-26T18:26:54Z

    GEODE-487: Remove integrationTest and distributedTest from build task.
    
    This commit removes integrationTest and distributedTest tasks from the
    build task lifecycle. The tasks were associated with the 'check' task,
    which is in the 'build' lifecycle. After removing the tasks, only the
    'test' task is run when build task is executed.
    
    Tested by running 'clean build'
    
    Review available at https://reviews.apache.org/r/39664

commit 5412c641176327f8e48fbf9af69d64fe2e3bd2e7
Author: William Markito <wm...@pivotal.io>
Date:   2015-10-26T23:32:32Z

    GEODE-479 - Closing PR #23 bumping language level to Java 8

commit e4573db9333a7509d217be7372e3e3b90a978584
Author: Darrel Schneider <ds...@pivotal.io>
Date:   2015-10-27T23:26:59Z

    GEODE-411: fix suspect string on disconnect
    
    DistributionManager now has a isCloseInProgress method
    that checks both the volatile closeInProgress boolean
    and ask its DistributedSystem if it is disconnecting.
    This new method is used in places that decide if a warning
    should be logged.

commit f6bceefe37133e06ea8da2eda33ce4b49af4b357
Author: Hitesh Khamesra <hk...@pivotal.io>
Date:   2015-10-28T16:33:24Z

    GEODE-501. test no-ack region thus we need to wait before validating
    the results.

commit e7b0db1923d35acc0bea8c0e62cf6ba7b714a1e8
Author: Sai Boorlagadda <sb...@pivotal.io>
Date:   2015-10-28T18:37:49Z

    Fixes GEODE-485.
    * Loggin at debug level on a SocketException while closing the connection to avoid any suspect strings written to logs.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode pull request: Fixes GEODE-485.

Posted by upthewaterspout <gi...@git.apache.org>.

Github user upthewaterspout commented on the pull request:

    https://github.com/apache/incubator-geode/pull/26#issuecomment-151952966
  
    The changes in your last commit look good. But something is odd with this PR, you have some commits from develop that have been cherry picked onto your branch.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode pull request: Fixes GEODE-485.

Posted by sboorlagadda <gi...@git.apache.org>.

Github user sboorlagadda commented on the pull request:

    https://github.com/apache/incubator-geode/pull/26#issuecomment-151962659
  
    Thanks Dan. The branch is cleaned up reverting the cherry-picks.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode pull request: Fixes GEODE-485.

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-geode/pull/26


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode pull request: Fixes GEODE-485.

Posted by sboorlagadda <gi...@git.apache.org>.

Github user sboorlagadda commented on the pull request:

    https://github.com/apache/incubator-geode/pull/26#issuecomment-152623079
  
    Changed to info rather than debug.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---