/AbstractGatewaySenderEventProcessor.java
b41ace4
geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderAdvisor.java
97cfac7
Diff: https://reviews.apache.org/r/53745/diff/
Testing
---
Thanks,
xiaojian zhou
/main/java/org/apache/geode/internal/cache/wan/AbstractGatewaySenderEventProcessor.java
e953c0c
geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderAdvisor.java
ff36e06
Diff: https://reviews.apache.org/r/53745/diff/
Testing (updated)
---
Thanks,
xiaojian zhou
edgeOldThreadsPerVM = 1
nPutThreads=10 nPutKeyRange=10 version1=default version2=822
Thanks,
xiaojian zhou
/apache/geode/internal/cache/wan/GatewaySenderAdvisor.java
ff36e06
Diff: https://reviews.apache.org/r/53745/diff/
Testing
---
Thanks,
xiaojian zhou
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53401/#review154643
---
Ship it!
Ship It!
- xiaojian zhou
On Nov. 2, 2016, 8:14 p.m
+1
On Mon, Oct 31, 2016 at 4:12 PM, Jianxia Chen wrote:
> +1
>
> On Mon, Oct 31, 2016 at 2:47 PM, Anilkumar Gingade
> wrote:
>
> > +1
> >
> > On Fri, Oct 28, 2016 at 10:23 PM, Avinash Dongre
> > wrote:
> >
> > > +1
> > >
> > > On Sat,
pache.org/r/52759/#comment221549>
This should be 4 seconds
geode-core/src/test/java/org/apache/geode/internal/cache/ha/HARegionQueueJUnitTest.java
(line 467)
<https://reviews.apache.org/r/52759/#comment221552>
This should be >=, not >
- xiaojian zhou
On Oct. 11, 201
---
Thanks,
xiaojian zhou
GEODE-933 and GEODE-977 are not reproducible either after run 30+ times. So
they are not flaky and can be closed for now.
On Sat, Oct 1, 2016 at 11:30 PM, Xiaojian Zhou <gz...@pivotal.io> wrote:
> 1011, 1062, 1066, 1147 have been run 30+ times without reproduce. So it's
> not flaky
1011, 1062, 1066, 1147 have been run 30+ times without reproduce. So it's
not flaky. I think we can close them. If reproduced someday, we can
re-open.
On Sat, Oct 1, 2016 at 5:09 PM, Anthony Baker wrote:
> I reviewed a bunch of CI failures today. I closed out duplicates and
;
> (Updated Sept. 23, 2016, 9:58 p.m.)
>
>
> Review request for geode, Barry Oglesby, Eric Shu, Jason Huynh, Dan Smith,
> and xiaojian zhou.
>
>
> Repository: geode
>
>
> Description
> ---
>
> Before my changes:
> 1. peek ca
/GatewaySenderEventRemoteDispatcher.java
6beb0ee046d7e0cae5985b6430a390dd68f1351a
Diff: https://reviews.apache.org/r/52179/diff/
Testing
---
Thanks,
xiaojian zhou
://reviews.apache.org/r/51585/diff/
Testing
---
geode-lucene:precheckin
Thanks,
xiaojian zhou
-
geode-wan/src/main/java/com/gemstone/gemfire/internal/cache/wan/GatewaySenderEventRemoteDispatcher.java
2625ad2
Diff: https://reviews.apache.org/r/51567/diff/
Testing
---
Thanks,
xiaojian zhou
,
xiaojian zhou
/lucene/test/LuceneTestUtilities.java
da281ff
Diff: https://reviews.apache.org/r/50420/diff/
Testing
---
Thanks,
xiaojian zhou
---
Thanks,
xiaojian zhou
-lucene tests passed.
Thanks,
xiaojian zhou
/repository/IndexRepositoryImplPerformanceTest.java
3155aaf
geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/test/IndexRepositorySpy.java
0b66f55
Diff: https://reviews.apache.org/r/50315/diff/
Testing
---
added junit test and all the geode-lucene tests passed.
Thanks,
xiaojian
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49966/#review141994
---
Ship it!
Update and ship it.
- xiaojian zhou
On July 12
If you don't have authority to add the page into geode's wiki, you can send
it to me.
On Tue, Jul 12, 2016 at 9:58 AM, Xiaojian Zhou <gz...@pivotal.io> wrote:
> You can add a wiki page for geode with chinese characters. I just did some
> test.
>
>
> https://cwiki.apache.o
wrote:
> Yeah, xiaojian.
>
>
>
> 2016-07-12 23:53 GMT+08:00 Xiaojian Zhou [via Apache Geode (Incubating)
> Developers Forum] <ml-node+s70738n7128...@n6.nabble.com>:
>
> > Do you mean to publish into geode wiki or check into geode source code?
> >
> > On T
Do you mean chinese file name or content in a PDF or text file?
On Tue, Jul 12, 2016 at 3:28 AM, theseusyang wrote:
> Hi All,
>
> I have translated some chinese documents about geode installation,
> deployment,features.
> Now the main question is that How to put the
Do you mean to publish into geode wiki or check into geode source code?
On Tue, Jul 12, 2016 at 9:05 AM, Xiaojian Zhou <gz...@pivotal.io> wrote:
> Do you mean chinese file name or content in a PDF or text file?
>
> On Tue, Jul 12, 2016 at 3:28 AM, theseusyang <theseusy...@
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49772/#review141227
---
Ship it!
Ship It!
- xiaojian zhou
On July 7, 2016, 10:39
/java/com/gemstone/gemfire/cache/lucene/internal/LuceneQueryFactoryImpl.java
62b4f56
geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/LuceneQueriesIntegrationTest.java
988e4f5
Diff: https://reviews.apache.org/r/49481/diff/
Testing
---
Thanks,
xiaojian zhou
/gemfire/cache/lucene/LuceneQueriesBase.java
26cd4aa
Diff: https://reviews.apache.org/r/48965/diff/
Testing
---
Thanks,
xiaojian zhou
/gemstone/gemfire/cache/lucene/LuceneQueriesBase.java
26cd4aa
Diff: https://reviews.apache.org/r/48965/diff/
Testing
---
Thanks,
xiaojian zhou
geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/test/LuceneTestUtilities.java
0cf8953
geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/test/TestObject.java
a2b249e
Diff: https://reviews.apache.org/r/48701/diff/
Testing
---
Thanks,
xiaojian zhou
/LuceneTestUtilities.java
0cf8953
Diff: https://reviews.apache.org/r/48701/diff/
Testing
---
Thanks,
xiaojian zhou
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48432/#review137430
---
Ship it!
Ship It!
- xiaojian zhou
On June 9, 2016, 6:41 p.m
what's the difference btw supplier and sampler?
On Wed, Jun 1, 2016 at 4:12 PM, Dan Smith wrote:
> Hi,
>
> I'd like to add some new methods to the Statistics interface to compute
> statistics using callbacks. My original motivation for this is to make it
> easy to record
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48145/#review135848
---
Ship it!
Ship It!
- xiaojian zhou
On June 1, 2016, 6:34 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48138/#review135838
---
Ship it!
Ship It!
- xiaojian zhou
On June 1, 2016, 5:52 p.m
<dsm...@pivotal.io> wrote:
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48086/
>
> On May 31st, 2016, 11:01 p.m. UTC, *xiaojian zhou* wrote:
>
>
> geode-lucene/src/test/java/com/gemstone/gemfire/cach
IndexImpl
to do this function execution.
Then I can call:
myIndex.convertToFile(diskDir) without know the detail syntax of
FunctionService.
BTW, I think it can be shipped.
My above suggestion can be done in step2 after checked in current code.
- xiaojian zhou
On
to check if haContainer == null in
HARegionQueue.java?
- xiaojian zhou
On May 26, 2016, 8:34 p.m., Jianxia Chen wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
ence it in other classes, such as RepositoryManager.
- xiaojian zhou
On May 26, 2016, 6:26 p.m., Dan Smith wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47911/#review135037
---
Ship it!
Ship It!
- xiaojian zhou
On May 26, 2016, 6:22 p.m
/tier/sockets/CacheClientNotifier.java
(line 86)
<https://reviews.apache.org/r/47908/#comment28>
You should get rid of "synchronized" on the methods.
Others look good.
Fit it and ship it.
- xiaojian zhou
On May 26, 2016, 5:23 p.m., Ji
/lucene/internal/repository/serializer/PdxFieldMapperJUnitTest.java
278e818
geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/repository/serializer/ReflectionFieldMapperJUnitTest.java
c187022
Diff: https://reviews.apache.org/r/47803/diff/
Testing
---
Thanks,
xiaojian
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47675/#review134436
---
Ship it!
Ship It!
- xiaojian zhou
On May 20, 2016, 10:18
---
Thanks,
xiaojian zhou
-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/LuceneQueriesIntegrationTest.java
15f5747
geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/test/LuceneTestUtilities.java
571049c
Diff: https://reviews.apache.org/r/47712/diff/
Testing
---
Thanks,
xiaojian zhou
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47503/#review133694
---
Ship it!
Ship It!
- xiaojian zhou
On May 17, 2016, 11:58
://reviews.apache.org/r/47004/diff/
Testing
---
Thanks,
xiaojian zhou
/cache/lucene/LuceneQueriesBase.java
c467a18
geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/LuceneIndexImplJUnitTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47004/diff/
Testing
---
Thanks,
xiaojian zhou
for resources) in tomcat7.
That's good.
- xiaojian zhou
On May 13, 2016, 12:14 a.m., Dan Smith wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
.java
PRE-CREATION
Diff: https://reviews.apache.org/r/47254/diff/
Testing
---
Thanks,
xiaojian zhou
/wan/Simple2CacheServerDUnitTest.java
4fdd88d
Diff: https://reviews.apache.org/r/47220/diff/
Testing
---
Thanks,
xiaojian zhou
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47148/#review132530
---
Ship it!
Ship It!
- xiaojian zhou
On May 10, 2016, 1:13 a.m
://reviews.apache.org/r/47083/diff/
Testing
---
Thanks,
xiaojian zhou
/dunit/standalone/DUnitLauncher.java
be459b9
geode-core/src/test/java/com/gemstone/gemfire/test/dunit/standalone/ProcessManager.java
dd532c4
gradle/test.gradle 2dd7521
Diff: https://reviews.apache.org/r/47083/diff/
Testing
---
Thanks,
xiaojian zhou
/java/com/gemstone/gemfire/internal/cache/wan/Simple2CacheServerDUnitTest.java
684660b
Diff: https://reviews.apache.org/r/47046/diff/
Testing
---
Thanks,
xiaojian zhou
Diff: https://reviews.apache.org/r/47046/diff/
Testing
---
Thanks,
xiaojian zhou
/java/com/gemstone/gemfire/internal/cache/wan/Simple2CacheServerDUnitTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/46625/diff/
Testing
---
Thanks,
xiaojian zhou
/sockets/CacheClientNotifier.java
1ba2294
geode-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/CacheClientNotifierDUnitTest.java
8bf819c
Diff: https://reviews.apache.org/r/46625/diff/
Testing
---
Thanks,
xiaojian zhou
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46239/#review129067
---
Ship it!
Ship It!
- xiaojian zhou
On April 15, 2016, 12:16
+1
On Wed, Apr 13, 2016 at 3:55 PM, Dan Smith wrote:
> This is second release candidate for Apache Geode, version
> 1.0.0-incubating.M1.
>
> *** Please download, test and vote by Monday, Apr 18, 0800 PST
>
> If you are wondering why you didn't see RC1, it's because I caught a
geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/LuceneServiceImplJUnitTest.java
159fd46
Diff: https://reviews.apache.org/r/46065/diff/
Testing
---
Thanks,
xiaojian zhou
,
xiaojian zhou
ut it's ok either way.
It's good.
- xiaojian zhou
On April 7, 2016, 5:22 p.m., Jason Huynh wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://revi
new dunit test.
- precheckin
- regression tests:
Thanks,
xiaojian zhou
- regression tests:
Thanks,
xiaojian zhou
/CacheClientNotifierDUnitTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/45336/diff/
Testing
---
- introduced new dunit test.
- precheckin
- regression tests:
Thanks,
xiaojian zhou
Diff: https://reviews.apache.org/r/44601/diff/
Testing
---
Thanks,
xiaojian zhou
/gemstone/gemfire/internal/cache/partitioned/FetchEntriesMessage.java
fda15c4
geode-core/src/test/java/com/gemstone/gemfire/internal/cache/partitioned/FetchEntriesMessageJUnitTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/44540/diff/
Testing
---
Thanks,
xiaojian zhou
gradle/test.gradle d8af4ba
Diff: https://reviews.apache.org/r/44412/diff/
Testing
---
precheckin and some dunit tests
Thanks,
xiaojian zhou
/FetchEntriesMessage.java
0342164
Diff: https://reviews.apache.org/r/44306/diff/
Testing
---
precheckin
the failed test itself
Thanks,
xiaojian zhou
/test/java/com/gemstone/gemfire/test/fake/Fakes.java ffb4896
Diff: https://reviews.apache.org/r/44118/diff/
Testing
---
junit tests
Thanks,
xiaojian zhou
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43674/#review119533
---
Ship it!
Ship It!
- xiaojian zhou
On Feb. 17, 2016, 8:44
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43129/#review117577
---
Ship it!
Ship It!
- xiaojian zhou
On Feb. 3, 2016, 1:51 a.m
diff, you did not remove
"testOutput {...}" section from open/build.gradle. We still need it?
- xiaojian zhou
On Jan. 30, 2016, 5:45 p.m., Anthony Baker wrote:
>
> ---
> This is an automatically generated e-mail. To re
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42892/#review116687
---
Ship it!
Ship It!
- xiaojian zhou
On Jan. 28, 2016, 1:47
/wan/concurrent/ConcurrentParallelGatewaySenderOperation_2_DUnitTest.java
(line 526)
<https://reviews.apache.org/r/42889/#comment177762>
I have been thinking of make this method more generic to be shared by all
the WAN test cases and AEQ test cases.
Others look good.
- xiaojia
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42886/#review116680
---
Ship it!
Ship It!
- xiaojian zhou
On Jan. 28, 2016, 12:46
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42885/#review116669
---
Ship it!
Ship It!
- xiaojian zhou
On Jan. 28, 2016, 12:43
/wan/WANTestBase.java
(line 2165)
<https://reviews.apache.org/r/42822/#comment177495>
There is a pauseSender() and pauseSenderAndWaitForDispatcherToPause(). You
just made the 2 methods identical.
Was that purposely using pauseSender() here?
Others look good.
- xiaojia
+1
It saved a lot of trouble in tracing the method.
And introduced a simple interface can make such a big difference.
On Tue, Jan 26, 2016 at 3:22 PM, Kenneth Howe wrote:
> +1
>
> Ken
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42039/#review115467
---
Ship it!
Ship It!
- xiaojian zhou
On Jan. 15, 2016, 1:55 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41492/#review115474
---
Ship it!
Ship It!
- xiaojian zhou
On Jan. 16, 2016, 1:45 a.m
I am wrong. Just so that you can take a look. Thanks!
>
>
> - Jinmei
>
> On January 18th, 2016, 6:29 p.m. UTC, Bruce Schuchardt wrote:
> Review request for geode, Dan Smith and xiaojian zhou.
> By Bruce Schuchardt.
>
> *Updated Jan. 18, 2016, 6:29 p.m.*
> *Reposit
java/com/gemstone/gemfire/internal/cache/DistributedRegion.java
92b585a
Diff: https://reviews.apache.org/r/42099/diff/
Testing
---
The in-house hydra test to reproduce it.
precheckin.
A lot of wan hydra tests.
Thanks,
xiaojian zhou
f: https://reviews.apache.org/r/42099/diff/
Testing
---
The in-house hydra test to reproduce it.
precheckin.
A lot of wan hydra tests.
Thanks,
xiaojian zhou
/internal/beans/stats/StatsKey.java
<https://reviews.apache.org/r/41492/#comment170910>
You removed this stats "eventQueueSize", vsd cannot show it any more.
How about fix the size in stats too?
- xiaojian zhou
On Dec. 17, 2015, 2:43 a.m.,
t; seems like there might be other code that is expecting to find a region
> that previously existed, and the only reason that it is gone is because of
> a shut down all.
>
> - Dan Smith
>
>
> On Dec. 14, 2015, 1:06 a.m., xiaojian zhou wrote:
> >
> >
/PRClientServerRegionFunctionExecutionSingleHopDUnitTest.java
c0b0d3c
Diff: https://reviews.apache.org/r/39763/diff/
Testing
---
the dunit test case
Thanks,
xiaojian zhou
crashed.
Scheduling recovery.
So the disconnecting is expected.
- xiaojian
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39763/#review104474
---
: https://reviews.apache.org/r/39747/diff/
Testing
---
the dunit test
Thanks,
xiaojian zhou
for DR.
- xiaojian zhou
On Oct. 14, 2015, 9:21 p.m., Dan Smith wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39020/#review101605
---
Ship it!
Ship It!
- xiaojian zhou
On Oct. 5, 2015, 9:40 p.m
current DS is
disconnecting.
Diffs (updated)
-
gemfire-core/src/main/java/com/gemstone/gemfire/distributed/DistributedSystem.java
b7b2cd8
Diff: https://reviews.apache.org/r/38572/diff/
Testing
---
precheckin
Thanks,
xiaojian zhou
java
b7b2cd8
Diff: https://reviews.apache.org/r/38572/diff/
Testing
---
precheckin
Thanks,
xiaojian zhou
-
gemfire-core/src/main/java/com/gemstone/gemfire/distributed/DistributedSystem.java
b7b2cd8
Diff: https://reviews.apache.org/r/38572/diff/
Testing
---
precheckin
Thanks,
xiaojian zhou
No. ALL does not count. I remembered that I have to manually specify the
range.
On Mon, Aug 17, 2015 at 6:06 PM, Roman Shaposhnik ro...@shaposhnik.org
wrote:
On Mon, Aug 17, 2015 at 5:40 PM, Xiaojian Zhou gz...@pivotal.io wrote:
This error msg actually is not the root cause of the issue
95 matches
Mail list logo