[jira] [Commented] (GEODE-2428) Add support for LinkedHashMap in DataSerializer

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885643#comment-15885643 ] ASF subversion and git services commented on GEODE-2428: Commit a8

[jira] [Commented] (GEODE-2428) Add support for LinkedHashMap in DataSerializer

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885642#comment-15885642 ] ASF subversion and git services commented on GEODE-2428: Commit a8

[GitHub] geode pull request #388: GEODE-2428: Adding support of Serialization/deseria...

2017-02-27 Thread davinash
Github user davinash closed the pull request at: https://github.com/apache/geode/pull/388 --- 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 ena

[GitHub] geode issue #388: GEODE-2428: Adding support of Serialization/deserializatio...

2017-02-27 Thread davinash
Github user davinash commented on the issue: https://github.com/apache/geode/pull/388 Closing, merged to develop --- 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 wishe

[jira] [Commented] (GEODE-2428) Add support for LinkedHashMap in DataSerializer

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885650#comment-15885650 ] ASF GitHub Bot commented on GEODE-2428: --- Github user davinash commented on the issue

[jira] [Commented] (GEODE-2428) Add support for LinkedHashMap in DataSerializer

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885651#comment-15885651 ] ASF GitHub Bot commented on GEODE-2428: --- Github user davinash closed the pull reques

[jira] [Resolved] (GEODE-2428) Add support for LinkedHashMap in DataSerializer

2017-02-27 Thread Avinash Dongre (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Avinash Dongre resolved GEODE-2428. --- Resolution: Fixed > Add support for LinkedHashMap in DataSerializer >

[jira] [Updated] (GEODE-2428) Add support for LinkedHashMap in DataSerializer

2017-02-27 Thread Avinash Dongre (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Avinash Dongre updated GEODE-2428: -- Fix Version/s: 1.2.0 > Add support for LinkedHashMap in DataSerializer > ---

[jira] [Commented] (GEODE-1887) Client PROXY region should delegate all operations to server

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885767#comment-15885767 ] ASF subversion and git services commented on GEODE-1887: Commit f2

[jira] [Commented] (GEODE-1887) Client PROXY region should delegate all operations to server

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885769#comment-15885769 ] ASF subversion and git services commented on GEODE-1887: Commit f2

[jira] [Commented] (GEODE-1887) Client PROXY region should delegate all operations to server

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885768#comment-15885768 ] ASF subversion and git services commented on GEODE-1887: Commit f2

[jira] [Commented] (GEODE-1887) Client PROXY region should delegate all operations to server

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885766#comment-15885766 ] ASF subversion and git services commented on GEODE-1887: Commit f2

[jira] [Commented] (GEODE-1887) Client PROXY region should delegate all operations to server

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885770#comment-15885770 ] ASF subversion and git services commented on GEODE-1887: Commit f2

[GitHub] geode issue #245: native-client-software-grant - ClientMetadata::getServerLo...

2017-02-27 Thread doribd
Github user doribd commented on the issue: https://github.com/apache/geode/pull/245 ok thanks, understood now. --- 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

jgroups 4.0.0.Final released

2017-02-27 Thread Dor Ben Dov
Hi, Have it being considered switching from 3.6.10 Final to 4.0.0 Final release. From Bella's blog it seems that the new version uses java 8 streams, lambda expressions as well as 120 issues that were fixed as well as optimizations. Regards, Dor This message and the information contained he

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Udo Kohlmeyer
I've quickly gone through the changes for the pull request. The most significant change of this pull request is that the collections that initially were regions are single collections (not distributed). That said, this is something that we've been discussing. The one thing that I wonder about

[GitHub] geode issue #404: Geode 2469

2017-02-27 Thread kohlmu-pivotal
Github user kohlmu-pivotal commented on the issue: https://github.com/apache/geode/pull/404 I like the fact that we are now dealing with collections on a single machine, rather than all the elements spread distributed. Would you possibly have some perf metrics in relation to larger co

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Wes Williams
>>Replicating a whole collection because of 1 change does not really make too much sense.<< I agree but won't delta replication prevent sending the entire collection across the wire? *Wes Williams | Pivotal Advisory **Data Engineer* 781.606.0325 http://pivotal.io/big-data/pivotal-gemfire On Mon,

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885970#comment-15885970 ] ASF subversion and git services commented on GEODE-2142: Commit 76

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885975#comment-15885975 ] ASF subversion and git services commented on GEODE-2142: Commit 7c

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885976#comment-15885976 ] ASF subversion and git services commented on GEODE-2142: Commit ea

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885974#comment-15885974 ] ASF subversion and git services commented on GEODE-2142: Commit fd

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885977#comment-15885977 ] ASF subversion and git services commented on GEODE-2142: Commit cd

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885973#comment-15885973 ] ASF subversion and git services commented on GEODE-2142: Commit b3

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885972#comment-15885972 ] ASF subversion and git services commented on GEODE-2142: Commit a2

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885980#comment-15885980 ] ASF subversion and git services commented on GEODE-2142: Commit f2

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885979#comment-15885979 ] ASF subversion and git services commented on GEODE-2142: Commit ab

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread Udo Kohlmeyer (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885981#comment-15885981 ] Udo Kohlmeyer commented on GEODE-2142: -- Replaced org.json implementation with an open

[jira] [Commented] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885978#comment-15885978 ] ASF subversion and git services commented on GEODE-2142: Commit e2

[jira] [Updated] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread Udo Kohlmeyer (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Udo Kohlmeyer updated GEODE-2142: - Fix Version/s: 1.2.0 > Remove JSON.org dependency > -- > >

[jira] [Resolved] (GEODE-2142) Remove JSON.org dependency

2017-02-27 Thread Udo Kohlmeyer (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Udo Kohlmeyer resolved GEODE-2142. -- Resolution: Fixed Assignee: Udo Kohlmeyer > Remove JSON.org dependency > -

[GitHub] geode-native issue #39: Add rat checks to travis-ci build

2017-02-27 Thread PivotalSarge
Github user PivotalSarge commented on the issue: https://github.com/apache/geode-native/pull/39 Where is .ratignore? I don't see it. --- 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 featu

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Udo Kohlmeyer
Delta could provide us a mechanism to replicate only what is required. I wonder if we could not create a simple operation replication framework. Rather than writing a potential large amounts of code for delta, we replicate only the operation. On 2/27/17 07:18, Wes Williams wrote: Replicatin

[jira] [Created] (GEODE-2552) Replace NanoTimer with std::chrono / std::this_thread::sleep_for

2017-02-27 Thread Jacob S. Barrett (JIRA)
Jacob S. Barrett created GEODE-2552: --- Summary: Replace NanoTimer with std::chrono / std::this_thread::sleep_for Key: GEODE-2552 URL: https://issues.apache.org/jira/browse/GEODE-2552 Project: Geode

Build failed in Jenkins: Geode-nightly #761

2017-02-27 Thread Apache Jenkins Server
See -- [...truncated 172.02 KB...] :geode-json:spotlessCheck :geode-json:test UP-TO-DATE :geode-json:check :geode-json:build :geode-json:distributedTest UP-TO-DATE :geode-json:flakyTest UP-TO

[GitHub] geode-native issue #39: Add rat checks to travis-ci build

2017-02-27 Thread echobravopapa
Github user echobravopapa commented on the issue: https://github.com/apache/geode-native/pull/39 @PivotalSarge .ratignore is in the base dir of geode-native --- 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

Re: Review Request 57070: LuceneServiceImpl should throw CacheClosedException if found cache is null

2017-02-27 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57070/#review166884 --- I don't think this is the appropriate thing to do here. The user h

[GitHub] geode-native pull request #40: GEODE-2552: Remove NanoTimer

2017-02-27 Thread pivotal-jbarrett
GitHub user pivotal-jbarrett opened a pull request: https://github.com/apache/geode-native/pull/40 GEODE-2552: Remove NanoTimer Please focus on commits e541e31 ... daf4234, others are artifacts of unpulled requests. You can merge this pull request into a Git repository by running:

[GitHub] geode-native issue #39: Add rat checks to travis-ci build

2017-02-27 Thread PivotalSarge
Github user PivotalSarge commented on the issue: https://github.com/apache/geode-native/pull/39 Ah. `find` does not behave like I think it does with respect to finding dot files in the current directory. --- If your project is set up for it, you can reply to this email and have your

[jira] [Commented] (GEODE-2552) Replace NanoTimer with std::chrono / std::this_thread::sleep_for

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886045#comment-15886045 ] ASF GitHub Bot commented on GEODE-2552: --- GitHub user pivotal-jbarrett opened a pull

[GitHub] geode-native issue #39: Add rat checks to travis-ci build

2017-02-27 Thread echobravopapa
Github user echobravopapa commented on the issue: https://github.com/apache/geode-native/pull/39 I quoted the target, suspecting find would not like it otherwise... --- 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

[GitHub] geode pull request #390: [GEODE-1887] #comment Fix Issue Client PROXY region...

2017-02-27 Thread davinash
Github user davinash closed the pull request at: https://github.com/apache/geode/pull/390 --- 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 ena

[jira] [Commented] (GEODE-1887) Client PROXY region should delegate all operations to server

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886066#comment-15886066 ] ASF GitHub Bot commented on GEODE-1887: --- Github user davinash closed the pull reques

[GitHub] geode-native issue #40: GEODE-2552: Remove NanoTimer

2017-02-27 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on the issue: https://github.com/apache/geode-native/pull/40 > Would adding const to variables like spentWorking and sleepDuration help the compiler generate more efficient code? I doubt the compilers would care since it will know the sco

[jira] [Commented] (GEODE-2552) Replace NanoTimer with std::chrono / std::this_thread::sleep_for

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886110#comment-15886110 ] ASF GitHub Bot commented on GEODE-2552: --- Github user pivotal-jbarrett commented on t

[GitHub] geode-native issue #40: GEODE-2552: Remove NanoTimer

2017-02-27 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on the issue: https://github.com/apache/geode-native/pull/40 @PivotalSarge > Why the switch to ACE_OS::localtime() from localtime()? `localtime` AKA POSIX `localtime` is not thread safe. `ACE_OS::localtime` wraps the platform specifi

[jira] [Commented] (GEODE-2552) Replace NanoTimer with std::chrono / std::this_thread::sleep_for

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886116#comment-15886116 ] ASF GitHub Bot commented on GEODE-2552: --- Github user pivotal-jbarrett commented on t

[GitHub] geode-native issue #40: GEODE-2552: Remove NanoTimer

2017-02-27 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on the issue: https://github.com/apache/geode-native/pull/40 @PivotalSarge > Nice fix of the broken window of catching exceptions by value. If you enable `clang-tidy` you will see plenty of them. I am cleaning up lots of these bad th

[jira] [Commented] (GEODE-2552) Replace NanoTimer with std::chrono / std::this_thread::sleep_for

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886114#comment-15886114 ] ASF GitHub Bot commented on GEODE-2552: --- Github user pivotal-jbarrett commented on t

Re: Feature branch cleanup

2017-02-27 Thread Dan Smith
+1 -Dan On Fri, Feb 24, 2017 at 10:35 PM, Anthony Baker wrote: > > The following 100+ feature branches are related to closed bugs. I think > these can safely be deleted (the branch names aren’t exact but pretty > close): > > GEODE-1017: "Closed" > GEODE-1040: "Closed" > GEODE-1050: "Closed" >

[GitHub] geode-native pull request #37: GEODE-2531: Replace HostAsm::atomic with std:...

2017-02-27 Thread PivotalSarge
Github user PivotalSarge commented on a diff in the pull request: https://github.com/apache/geode-native/pull/37#discussion_r103254380 --- Diff: src/cppcache/include/geode/CacheStatistics.hpp --- @@ -102,8 +102,8 @@ class CPPCACHE_EXPORT CacheStatistics : public SharedBase {

[GitHub] geode-native pull request #37: GEODE-2531: Replace HostAsm::atomic with std:...

2017-02-27 Thread PivotalSarge
Github user PivotalSarge commented on a diff in the pull request: https://github.com/apache/geode-native/pull/37#discussion_r103257502 --- Diff: src/cppcache/src/statistics/AtomicStatisticsImpl.cpp --- @@ -330,7 +329,13 @@ double AtomicStatisticsImpl::_incDouble(int32_t offset, dou

[GitHub] geode-native pull request #37: GEODE-2531: Replace HostAsm::atomic with std:...

2017-02-27 Thread PivotalSarge
Github user PivotalSarge commented on a diff in the pull request: https://github.com/apache/geode-native/pull/37#discussion_r103255168 --- Diff: src/cppcache/include/geode/SharedBase.hpp --- @@ -56,11 +57,12 @@ class CPPCACHE_EXPORT SharedBase { protected: inline

[jira] [Commented] (GEODE-2531) Replace HostAsm::atomic* and AtomicInc with std::atomic

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886129#comment-15886129 ] ASF GitHub Bot commented on GEODE-2531: --- Github user PivotalSarge commented on a dif

[jira] [Commented] (GEODE-2515) CI Failure: BaseLineAndCompareQueryPerfJUnitTest.testPerformanceForRangeQueries FAILED failed with AssertionFailedError

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886132#comment-15886132 ] ASF subversion and git services commented on GEODE-2515: Commit 8f

[jira] [Commented] (GEODE-2531) Replace HostAsm::atomic* and AtomicInc with std::atomic

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886130#comment-15886130 ] ASF GitHub Bot commented on GEODE-2531: --- Github user PivotalSarge commented on a dif

[jira] [Commented] (GEODE-2531) Replace HostAsm::atomic* and AtomicInc with std::atomic

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886131#comment-15886131 ] ASF GitHub Bot commented on GEODE-2531: --- Github user PivotalSarge commented on a dif

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Michael Stolz
It does seem like the operations will often be much smaller than the data they are operating on. It is almost the classic "move the code to the data" pattern. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: +1-631-835-4771 On Mon, Feb 27, 2017 at 10:51 AM, Udo Kohlmeyer wrote:

[jira] [Resolved] (GEODE-2515) CI Failure: BaseLineAndCompareQueryPerfJUnitTest.testPerformanceForRangeQueries FAILED failed with AssertionFailedError

2017-02-27 Thread Dan Smith (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2515?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dan Smith resolved GEODE-2515. -- Resolution: Fixed Assignee: Dan Smith Fix Version/s: 1.2.0 > CI Failure: > BaseLineAndC

[GitHub] geode issue #406: GEODE-2526: Enhance log statement to include ResourceTypeN...

2017-02-27 Thread smanvi-pivotal
Github user smanvi-pivotal commented on the issue: https://github.com/apache/geode/pull/406 @metatype Thank you. I just pushed a new commit with a small refactor. Travis build is in progress. I did run `./gradlew spotlessApply` and `./gradlew build` was successful. --- If your p

[jira] [Commented] (GEODE-2526) Enhance log statement in StatsArchiveReader

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2526?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886144#comment-15886144 ] ASF GitHub Bot commented on GEODE-2526: --- Github user smanvi-pivotal commented on the

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103252234 --- Diff: src/cppcache/src/MapSegment.hpp --- @@ -164,9 +166,9 @@ class CPPCACHE_EXPORT MapSegment { m_entryFactory->newMapEntry(key, newEntry)

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103252978 --- Diff: src/cppcache/integration-test/testSpinLock.cpp --- @@ -20,48 +20,44 @@ #include "fw_dunit.hpp" #include +#include +#

[GitHub] geode-native issue #34: GEODE-2513 Rebrand Delta Propagation section of

2017-02-27 Thread davebarnes97
Github user davebarnes97 commented on the issue: https://github.com/apache/geode-native/pull/34 +1 --- 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 th

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103257643 --- Diff: src/cppcache/src/LRUList.cpp --- @@ -15,12 +15,18 @@ * limitations under the License. */ #include "LRUList.hpp" -#include

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103256958 --- Diff: src/cppcache/src/CqQueryVsdStats.cpp --- @@ -108,21 +98,21 @@ CqQueryStatType::CqQueryStatType() ///

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103262976 --- Diff: src/cppcache/src/MapSegment.hpp --- @@ -41,6 +40,9 @@ #include #include "TombstoneList.hpp" #include + +#include "uti

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886161#comment-15886161 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886169#comment-15886169 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[jira] [Commented] (GEODE-2513) Geode Native docs: rebrand to match open-source software

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886172#comment-15886172 ] ASF GitHub Bot commented on GEODE-2513: --- Github user davebarnes97 commented on the i

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886167#comment-15886167 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886168#comment-15886168 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886163#comment-15886163 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103256152 --- Diff: src/cppcache/src/CqQueryVsdStats.cpp --- @@ -20,43 +20,38 @@ #include "CqQueryVsdStats.hpp" //#include "StatisticsFactory.hpp"

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886166#comment-15886166 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103258922 --- Diff: src/cppcache/src/MapSegment.cpp --- @@ -19,18 +19,22 @@ #include "TrackedMapEntry.hpp" #include "RegionInternal.hpp" #include "

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886165#comment-15886165 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886162#comment-15886162 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886170#comment-15886170 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886171#comment-15886171 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103258079 --- Diff: src/cppcache/src/LRUList.cpp --- @@ -96,33 +102,38 @@ void LRUList::getLRUEntry(LRUListEntryPtr& result) { template typename

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103251295 --- Diff: src/tests/cpp/security/Security.cpp --- @@ -1014,12 +1016,15 @@ int32_t Security::doEntryOperations() { reinterpret_cast(

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103251646 --- Diff: src/cppcache/src/util/concurrent/spinlock_mutex.hpp --- @@ -15,23 +15,37 @@ * limitations under the License. */ -#includ

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread dgkimura
Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103252510 --- Diff: src/cppcache/integration-test/testSpinLock.cpp --- @@ -20,48 +20,44 @@ #include "fw_dunit.hpp" #include +#include +#

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886164#comment-15886164 ] ASF GitHub Bot commented on GEODE-2494: --- Github user dgkimura commented on a diff in

[GitHub] geode pull request #407: GEODE-2550 Improve README and BUILDING

2017-02-27 Thread karensmolermiller
Github user karensmolermiller commented on a diff in the pull request: https://github.com/apache/geode/pull/407#discussion_r103267882 --- Diff: README.md --- @@ -1,102 +1,191 @@ [https://geode.apache.org/img/apache_geode_logo.png"; align="center"/>](http://geode.apache.org)

[jira] [Commented] (GEODE-2550) Improve README

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2550?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886209#comment-15886209 ] ASF GitHub Bot commented on GEODE-2550: --- Github user karensmolermiller commented on

[jira] [Commented] (GEODE-2545) NPE during lucene query execution when cache is closing or region is destroyed

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886232#comment-15886232 ] ASF subversion and git services commented on GEODE-2545: Commit c4

[jira] [Resolved] (GEODE-2545) NPE during lucene query execution when cache is closing or region is destroyed

2017-02-27 Thread Jason Huynh (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh resolved GEODE-2545. Resolution: Fixed Fix Version/s: 1.2.0 > NPE during lucene query execution when cache is clos

[jira] [Resolved] (GEODE-2543) Remove incorrectly marked deprecated methods from LuceneService and LuceneQueryFactory api

2017-02-27 Thread Jason Huynh (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2543?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh resolved GEODE-2543. Resolution: Fixed Fix Version/s: 1.2.0 > Remove incorrectly marked deprecated methods from Lu

[jira] [Resolved] (GEODE-2478) Replace gf with geode

2017-02-27 Thread Addison (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2478?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Addison resolved GEODE-2478. Resolution: Fixed > Replace gf with geode > - > > Key: GEODE-2478 >

[jira] [Resolved] (GEODE-2480) Need to specify correct version of CMake required for building Native Client

2017-02-27 Thread Addison (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2480?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Addison resolved GEODE-2480. Resolution: Fixed > Need to specify correct version of CMake required for building Native Client > -

[GitHub] geode-native pull request #39: Add rat checks to travis-ci build

2017-02-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode-native/pull/39 --- 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

[jira] [Commented] (GEODE-2404) Add API to destroy a region containing lucene indexes

2017-02-27 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886248#comment-15886248 ] ASF subversion and git services commented on GEODE-2404: Commit 11

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Hitesh Khamesra
The simplicity of Redis API making this problem (delta update)most apparent. But,  I would imagine many Geode apps has a similar use case. -Hitesh From: Michael Stolz To: dev@geode.apache.org Sent: Monday, February 27, 2017 9:06 AM Subject: Re: [DISCUSS] changes to Redis implementatio

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread PivotalSarge
Github user PivotalSarge commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103279163 --- Diff: src/cppcache/src/LRUList.hpp --- @@ -20,32 +20,32 @@ * limitations under the License. */ +#include + #incl

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread PivotalSarge
Github user PivotalSarge commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103278690 --- Diff: src/cppcache/src/LRUList.cpp --- @@ -15,12 +15,18 @@ * limitations under the License. */ #include "LRUList.hpp" -#inc

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886286#comment-15886286 ] ASF GitHub Bot commented on GEODE-2494: --- Github user PivotalSarge commented on a dif

[jira] [Commented] (GEODE-2494) Replace SpinLock class with C++11 style BasicLockable class, spinlock_mutex.

2017-02-27 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886285#comment-15886285 ] ASF GitHub Bot commented on GEODE-2494: --- Github user PivotalSarge commented on a dif

[GitHub] geode-native pull request #36: GEODE-2494: Replace SpinLock with spinlock_mu...

2017-02-27 Thread PivotalSarge
Github user PivotalSarge commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103279475 --- Diff: src/cppcache/src/MapSegment.cpp --- @@ -316,16 +320,16 @@ GfErrType MapSegment::remove(const CacheableKeyPtr& key, CacheablePtr& oldValue

  1   2   >