[jira] [Commented] (GEODE-880) Remove unused classes

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

[GitHub] geode pull request #409: GEODE-880 Remove unused classes

2017-02-27 Thread metatype
GitHub user metatype opened a pull request: https://github.com/apache/geode/pull/409 GEODE-880 Remove unused classes @kirklund I think this should close out GEODE-880. You can merge this pull request into a Git repository by running: $ git pull https://github.com/metatype/incub

[jira] [Updated] (GEODE-728) Rename Execution.withArgs to Execution.setArguments

2017-02-27 Thread Anthony Baker (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-728?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anthony Baker updated GEODE-728: Labels: (was: starter) > Rename Execution.withArgs to Execution.setArguments >

[jira] [Updated] (GEODE-1577) Unhelpful generic types on Execution.execute

2017-02-27 Thread Anthony Baker (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1577?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anthony Baker updated GEODE-1577: - Labels: (was: starter) > Unhelpful generic types on Execution.execute >

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

2017-02-27 Thread Anthony Baker (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2550?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anthony Baker resolved GEODE-2550. -- Resolution: Fixed Fix Version/s: 1.2.0 > Improve README > -- > >

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

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

[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=15887236#comment-15887236 ] ASF GitHub Bot commented on GEODE-2550: --- Github user asfgit closed the pull request

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

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

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

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

[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=15887047#comment-15887047 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

[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=15886996#comment-15886996 ] ASF GitHub Bot commented on GEODE-2531: --- Github user pivotal-jbarrett commented on a

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

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

[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=15886991#comment-15886991 ] ASF GitHub Bot commented on GEODE-2531: --- Github user pivotal-jbarrett commented on a

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

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

[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=15886980#comment-15886980 ] ASF GitHub Bot commented on GEODE-2531: --- Github user pivotal-jbarrett commented on a

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

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

[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=15886970#comment-15886970 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

2017-02-27 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103353683 --- Diff: src/cppcache/src/LRUEntriesMap.cpp --- @@ -20,6 +20,9 @@ #include "MapSegment.hpp" #include "CacheImpl.hpp" +#incl

[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=15886968#comment-15886968 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

[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=15886969#comment-15886969 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

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

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

2017-02-27 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103353568 --- Diff: src/cppcache/src/LRUList.hpp --- @@ -20,32 +20,32 @@ * limitations under the License. */ +#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=15886965#comment-15886965 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

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

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

2017-02-27 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103353400 --- Diff: src/cppcache/src/MapSegment.cpp --- @@ -19,18 +19,22 @@ #include "TrackedMapEntry.hpp" #include "RegionInternal.hpp" #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=15886962#comment-15886962 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

[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=15886957#comment-15886957 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

2017-02-27 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103352845 --- Diff: src/cppcache/src/LRUList.cpp --- @@ -15,12 +15,18 @@ * limitations under the License. */ #include "LRUList.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=15886956#comment-15886956 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

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

[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=15886954#comment-15886954 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

2017-02-27 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103352237 --- 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=15886952#comment-15886952 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

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

[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=15886950#comment-15886950 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

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

[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=15886948#comment-15886948 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

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

[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=15886945#comment-15886945 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

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

[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=15886929#comment-15886929 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

[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=15886928#comment-15886928 ] ASF GitHub Bot commented on GEODE-2494: --- Github user pivotal-jbarrett commented on a

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

2017-02-27 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103350612 --- Diff: src/cppcache/src/LRUEntriesMap.cpp --- @@ -297,7 +295,7 @@ GfErrType LRUEntriesMap::put(const CacheableKeyPtr& key, }

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

2017-02-27 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/36#discussion_r103350610 --- Diff: src/tests/cpp/security/Security.cpp --- @@ -41,16 +41,18 @@ #include "security/CredentialGenerator.hpp" -namespac

Re: Review Request 57064: GEODE-2460: update dependency versions

2017-02-27 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57064/#review166976 --- Ship it! Ship It! - Anthony Baker On Feb. 28, 2017, 12:03 a.

Re: Review Request 57064: GEODE-2460: update dependency versions

2017-02-27 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57064/ --- (Updated Feb. 28, 2017, 12:03 a.m.) Review request for geode, Anthony Baker, Ja

Re: Review Request 57064: GEODE-2460: update dependency versions

2017-02-27 Thread Kirk Lund
> On Feb. 27, 2017, 11:13 p.m., Kirk Lund wrote: > > gradle/dependency-versions.properties, line 62 > > > > > > I suspect this can be removed as an explicit dependency Dropped this from update. I'll remove it in GEOD

Re: Review Request 57064: GEODE-2460: update dependency versions

2017-02-27 Thread Kirk Lund
> On Feb. 27, 2017, 11:31 p.m., Anthony Baker wrote: > > Can you fix these references too? Thanks! > > > > ``` > > ~/code/incubator-geode (develop)$ git grep -i "Jackson Core 2.8.2" > > geode-assembly/src/main/dist/NOTICE:Jackson Core 2.8.2 > > geode-web-api/src/main/webapp/META-INF/NOTICE:Jack

[jira] [Commented] (GEODE-2538) Lucene query with values should not invoke cache loader

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

Re: Review Request 57064: GEODE-2460: update dependency versions

2017-02-27 Thread Kirk Lund
> On Feb. 27, 2017, 11:31 p.m., Anthony Baker wrote: > > Can you fix these references too? Thanks! > > > > ``` > > ~/code/incubator-geode (develop)$ git grep -i "Jackson Core 2.8.2" > > geode-assembly/src/main/dist/NOTICE:Jackson Core 2.8.2 > > geode-web-api/src/main/webapp/META-INF/NOTICE:Jack

Re: Review Request 57057: GEODE-2538: Don't invoke a cache loader when fetching values for a lucene query

2017-02-27 Thread Dan Smith
> On Feb. 27, 2017, 10:29 p.m., Barry Oglesby wrote: > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/results/LuceneGetPageFunction.java, > > line 75 > > > > > > I'm not sure if you want to deal

Re: Review Request 57064: GEODE-2460: update dependency versions

2017-02-27 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57064/#review166967 --- Can you fix these references too? Thanks! ``` ~/code/incubator-g

[jira] [Updated] (GEODE-2555) Region Management docs page refers to the wrong field

2017-02-27 Thread Galen O'Sullivan (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2555?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Galen O'Sullivan updated GEODE-2555: Priority: Minor (was: Major) > Region Management docs page refers to the wrong field >

[jira] [Updated] (GEODE-2555) Region Management docs page refers to the wrong field

2017-02-27 Thread Galen O'Sullivan (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2555?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Galen O'Sullivan updated GEODE-2555: Summary: Region Management docs page refers to the wrong field (was: Region Management refe

[jira] [Updated] (GEODE-2555) Region Management docs page refers to the wrong field

2017-02-27 Thread Galen O'Sullivan (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2555?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Galen O'Sullivan updated GEODE-2555: Affects Version/s: 1.1.0 > Region Management docs page refers to the wrong field > -

[jira] [Updated] (GEODE-2555) Region Management refers to the wrong field

2017-02-27 Thread Galen O'Sullivan (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2555?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Galen O'Sullivan updated GEODE-2555: Summary: Region Management refers to the wrong field (was: Region Management refers to a fi

[jira] [Created] (GEODE-2555) Region Management refers to a field that doesn't work.

2017-02-27 Thread Galen O'Sullivan (JIRA)
Galen O'Sullivan created GEODE-2555: --- Summary: Region Management refers to a field that doesn't work. Key: GEODE-2555 URL: https://issues.apache.org/jira/browse/GEODE-2555 Project: Geode Is

[jira] [Updated] (GEODE-2553) Lucene search hangs on recreated region with no data

2017-02-27 Thread Diane Hardman (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2553?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Diane Hardman updated GEODE-2553: - Attachment: server50505.log This is the server logfile generated by GemFire. > Lucene search hang

[jira] [Updated] (GEODE-2553) Lucene search hangs on recreated region with no data

2017-02-27 Thread Diane Hardman (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2553?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Diane Hardman updated GEODE-2553: - Attachment: stack.log This is the stack trace acquired by running jstack on the PID for the server

[jira] [Created] (GEODE-2554) Geode incubator docs are still up

2017-02-27 Thread Galen O'Sullivan (JIRA)
Galen O'Sullivan created GEODE-2554: --- Summary: Geode incubator docs are still up Key: GEODE-2554 URL: https://issues.apache.org/jira/browse/GEODE-2554 Project: Geode Issue Type: Bug

[jira] [Created] (GEODE-2553) Lucene search hangs on recreated region with no data

2017-02-27 Thread Diane Hardman (JIRA)
Diane Hardman created GEODE-2553: Summary: Lucene search hangs on recreated region with no data Key: GEODE-2553 URL: https://issues.apache.org/jira/browse/GEODE-2553 Project: Geode Issue Type

Re: Review Request 57064: GEODE-2460: update dependency versions

2017-02-27 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57064/#review166961 --- gradle/dependency-versions.properties (line 62)

Review Request 57064: GEODE-2460: update dependency versions

2017-02-27 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57064/ --- Review request for geode, Anthony Baker, Jared Stewart, Mark Bretl, Udo Kohlmeye

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #485 was SUCCESSFUL (with 1679 tests). Change made by John Blum.

2017-02-27 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #485 was successful (rerun once). --- This build was rerun by John Blum. 1681 tests in total. https://build.sprin

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #486 was SUCCESSFUL (with 1679 tests)

2017-02-27 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #486 was successful. --- Scheduled 1681 tests in total. https://build.spring.io/browse/SGF-NAG-486/

Re: Review Request 57057: GEODE-2538: Don't invoke a cache loader when fetching values for a lucene query

2017-02-27 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57057/#review166946 --- geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal

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

2017-02-27 Thread Darrel Schneider (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Darrel Schneider reopened GEODE-2428: - I think the javadocs on these new methods need to make two more things clear: 1. readLinkedH

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

2017-02-27 Thread Barry Oglesby (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2404?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Barry Oglesby resolved GEODE-2404. -- Resolution: Fixed Fix Version/s: 1.2.0 > Add API to destroy a region containing lucene in

Re: Review Request 57057: GEODE-2538: Don't invoke a cache loader when fetching values for a lucene query

2017-02-27 Thread nabarun nag
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57057/#review166949 --- Ship it! Ship It! - nabarun nag On Feb. 25, 2017, 1:05 a.m.,

Re: Review Request 57057: GEODE-2538: Don't invoke a cache loader when fetching values for a lucene query

2017-02-27 Thread nabarun nag
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57057/#review166948 --- geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Dan Smith
> > Given that the feature is still labeled experimental, do our backwards > compatibility constraints apply? > Actually, it doesn't look like it is marked with @Experimental or is described as experimental in the docs. That said, I think maybe it *should* have been marked as experimental because

Re: Review Request 57057: GEODE-2538: Don't invoke a cache loader when fetching values for a lucene query

2017-02-27 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57057/#review166943 --- Fix it, then Ship it! Ship It! geode-lucene/src/main/java/org

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Dan Smith
> > Stick the operation+params into the EntryEventImpl that follows the > operation through the system and in the BucketRegion subclass pull out that > data and send it instead of the event's newValue. Yeah, but that only works if your events are always played in the same order. In the case of GI

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Anthony Baker
Given that the feature is still labeled experimental, do our backwards compatibility constraints apply? Anthony > On Feb 27, 2017, at 12:10 PM, Swapnil Bawaskar wrote: > > Accepting this pull request as-is will break backwards compatibility. I > think if the new behavior is desired, it should

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Swapnil Bawaskar
Accepting this pull request as-is will break backwards compatibility. I think if the new behavior is desired, it should be based on some configuration. On Mon, Feb 27, 2017 at 11:40 AM Bruce Schuchardt wrote: > I think in this case it would be replacing the messages used to > replicate changes w

[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=15886395#comment-15886395 ] ASF GitHub Bot commented on GEODE-2513: --- Github user karensmolermiller closed the pu

[GitHub] geode-native pull request #34: GEODE-2513 Rebrand Delta Propagation section ...

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

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

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

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

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

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

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

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Bruce Schuchardt
I think in this case it would be replacing the messages used to replicate changes with ones that send the operation + parameters instead of the modified entry's new value. That could be done by creating a new subclass of BucketRegion. Stick the operation+params into the EntryEventImpl that fo

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Real Wes
I'm not following what a "simple operation replication framework" is and how it applies to the Redis API. If you replicate operations, you still need to update the data at some point, causing a synchronous replication event so as to provide HA. What is, in more detail, a "simple operation replic

Re: [DISCUSS] changes to Redis implementation

2017-02-27 Thread Bruce Schuchardt
What I like about using the existing delta-prop mechanism is it will also extend to client subscriptions & WAN. It would take a lot of work to propagate Redis commands through those paths. Le 2/27/2017 à 10:35 AM, Hitesh Khamesra a écrit : The simplicity of Redis API making this problem (delt

[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=15886303#comment-15886303 ] 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_r103281152 --- Diff: src/tests/cpp/security/Security.cpp --- @@ -41,16 +41,18 @@ #include "security/CredentialGenerator.hpp" -namespace Fw

[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=15886294#comment-15886294 ] 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_r103280043 --- Diff: src/cppcache/src/LRUEntriesMap.cpp --- @@ -297,7 +295,7 @@ GfErrType LRUEntriesMap::put(const CacheableKeyPtr& key, }

[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=15886295#comment-15886295 ] 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_r103280113 --- Diff: src/cppcache/src/LRUEntriesMap.cpp --- @@ -20,6 +20,9 @@ #include "MapSegment.hpp" #include "CacheImpl.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=15886292#comment-15886292 ] 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=15886291#comment-15886291 ] 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_r103279753 --- Diff: src/cppcache/src/MapSegment.hpp --- @@ -164,9 +166,9 @@ class CPPCACHE_EXPORT MapSegment { m_entryFactory->newMapEntry(key, newEn

[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

[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

[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

[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

[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

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

[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

[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

  1   2   >