The maven repository for 1.8.0 seems to be missing the 1.8.0 source jars.
This means people using an IDE can't download these jars automatically to
see the geode source in their IDE.
Specifically, we see the following in 1.8 vs 1.7:
https://repo.maven.apache.org/maven2/org/apache/geode/geode-core/
Diff: https://reviews.apache.org/r/59863/diff/1/
Testing
---
Precheckin started
Thanks,
Brian Rowe
dated)
-
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/AcceptorImpl.java
9658f98da
Diff: https://reviews.apache.org/r/59863/diff/2/
Changes: https://reviews.apache.org/r/59863/diff/1-2/
Testing
---
Precheckin started
Thanks,
Brian Rowe
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59925/#review177371
---
Ship it!
Ship It!
- Brian Rowe
On June 8, 2017, 6:36 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59894/#review177374
---
Ship it!
Ship It!
- Brian Rowe
On June 7, 2017, 9:56 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59896/#review177375
---
Ship it!
Ship It!
- Brian Rowe
On June 7, 2017, 10:02 p.m
and LegacyServerConnection,
but don't have any great alternative suggestion at the moment.
- Brian Rowe
On June 16, 2017, 5:14 p.m., Galen O'Sullivan wrote:
>
> ---
> This is an automatically generated e-mail. To re
istry/CodecRegistryJUnitTest.java
PRE-CREATION
gradle/rat.gradle 1bea5843b
settings.gradle c0fdb6e4f
Diff: https://reviews.apache.org/r/60217/diff/1/
Testing
---
Precheckin in progress. Unit tests added for all new classes, integration test
added for entire module.
Thanks,
Brian Rowe
fixed
settings.gradle
Line 39 (original), 39 (patched)
<https://reviews.apache.org/r/60217/#comment252354>
Alright, I'll leave that for that batch of work then
- Brian Rowe
On June 20, 2017, 12:17 a.m., Brian Rowe wrote:
>
> -
Testing
---
Precheckin in progress. Unit tests added for all new classes, integration test
added for entire module.
Thanks,
Brian Rowe
d for entire module.
Thanks,
Brian Rowe
checkin in progress.
Thanks,
Brian Rowe
odule test, precheckin in progress.
Thanks,
Brian Rowe
org/r/60312/#comment253397>
Typo, s/loators/locators
geode-core/src/main/java/org/apache/geode/internal/cache/PoolFactoryImpl.java
Lines 308 (patched)
<https://reviews.apache.org/r/60312/#comment253399>
remove this
- Brian
iff: https://reviews.apache.org/r/60486/diff/1/
Testing
---
Added unit test verifying hanlder behavior.
Added integration test verifying module correctness for getRegion.
Running precheckin.
Thanks,
Brian Rowe
e-assembly/build.gradle 39bb54215
Diff: https://reviews.apache.org/r/60507/diff/1/
Testing
---
Thanks,
Brian Rowe
sting
---
This change is just adding a test.
Thanks,
Brian Rowe
Regions, which also seems like an improvement).
- Brian Rowe
On June 28, 2017, 11:06 p.m., Bruce Schuchardt wrote:
>
> ---
> This is an automatically generated e-mai
/60451/diff/1/?file=1763381#file1763381line46>
> >
> > This is too specific... What about all the other encoding types?
> > Int,long,short,byte?
We discussed templatizing this to make it more generally useful, but decided
that since this is a te
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60550/#review179374
---
Ship it!
Ship It!
- Brian Rowe
On June 29, 2017, 9:39 p.m
dd72a190e
Diff: https://reviews.apache.org/r/60629/diff/1/
Testing
---
Protobuf tests impacted by these changes have been refactored to check for
error responses.
Thanks,
Brian Rowe
8:45 p.m., Udo Kohlmeyer wrote:
> > geode-protobuf/src/main/proto/region_API.proto
> > Line 102 (original), 109 (patched)
> > <https://reviews.apache.org/r/60629/diff/1/?file=1768757#file1768757line109>
> >
> > As raised earlier, how about `GetAvailableReg
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60629/#review179796
---
On July 3, 2017, 11:40 p.m., Brian Rowe wrote:
>
> ---
gt; This has nothing to do with the ProtobufOpsProcessor. Maybe the builder
> > can be passed in.
>
> Brian Rowe wrote:
> Hmm, I'm not sure I agree. The ProtobufOpsProcessor has to generate the
> ClientProtocol.Response containing the response from the operation ha
`ErrorMessage`, that would contain the `exceptionCode`
> > and `errorMessage`.
This has been removed.
- Brian
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60486/#r
/test/resources/org/apache/geode/protocol/default.keystore
PRE-CREATION
Diff: https://reviews.apache.org/r/60526/diff/2/
Changes: https://reviews.apache.org/r/60526/diff/1-2/
Testing
---
This change is just adding a test.
Thanks,
Brian Rowe
ation test covering new operations
Thanks,
Brian Rowe
/diff/3/
Changes: https://reviews.apache.org/r/60526/diff/2-3/
Testing
---
This change is just adding a test.
Thanks,
Brian Rowe
:
https://reviews.apache.org/r/60526/#review179943
---
On July 7, 2017, 10:16 p.m., Brian Rowe wrote:
>
> ---
> This is an automatically generated e-
iff/1/?file=1771594#file1771594line48>
> >
> > Empty putAll does nothing. I don't believe we should log an error.
Fixed.
- Brian
---
This is an automatically generated e-mail. To reply, visit:
https://rev
could we possibly use `addAllFailedKeys(java.lang.Iterable > org.apache.geode.protocol.protobuf.BasicTypes.EncodedValue> values)`
This got removed with the failed keys field.
- Brian
---
This is an automatically generated e-ma
tobuf/operations/PutAllRequestOperationHandlerJUnitTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/60718/diff/3/
Changes: https://reviews.apache.org/r/60718/diff/2-3/
Testing
---
Added unit tests for new operation handlers
Added integration test covering new operations
Thanks,
Brian Rowe
https://reviews.apache.org/r/60718/diff/4/
Changes: https://reviews.apache.org/r/60718/diff/3-4/
Testing
---
Added unit tests for new operation handlers
Added integration test covering new operations
Thanks,
Brian Rowe
tobuf/operations/PutAllRequestOperationHandlerJUnitTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/60718/diff/2/
Changes: https://reviews.apache.org/r/60718/diff/1-2/
Testing
---
Added unit tests for new operation handlers
Added integration test covering new operations
Thanks,
Brian Rowe
application programmer supposed to determine what failed now?
> Without that information we may have rendered putAll unusable for some
> cases.
>
> --
> Mike Stolz
> Principal Engineer, GemFire Product Manager
> Mobile: +1-631-835-4771 <(631)%20835-4771>
>
> On Wed
s://reviews.apache.org/r/60718/#review180212
-------
On July 12, 2017, 6:27 p.m., Brian Rowe wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60718/
> --
--
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60718/#review180349
---
On July 12, 2017, 6:27 p.m., Brian Rowe wrote:
>
> -
/diff/1/
Testing
---
Thanks,
Brian Rowe
uf/operations/PutAllRequestOperationHandlerJUnitTest.java
> > Lines 68 (patched)
> > <https://reviews.apache.org/r/60718/diff/4/?file=1775044#file1775044line68>
> >
> > The serialization service is stateless. Would a spy make more sense
&
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60856/#review180500
---
Ship it!
Ship It!
- Brian Rowe
On July 13, 2017, 11:28 p.m
Can someone with the relevant karma please grant me edit permissions on
cwiki.apache.org/confluence. My username there is browe.
Thanks,
-Brian
Hello all,
I'm having some trouble assigning tickets to myself and closing them in
JIRA. Can someone grant me the appropriate permissions. My username is:
WireBaron
Thanks,
-Brian
ternal/tcp/ConnectionTableTest.java
Line 54 (original), 70 (patched)
<https://reviews.apache.org/r/61411/#comment258000>
Add a when(connection.isReceiverStopped()).thenReturn(false);
- Brian Rowe
On Aug. 3, 2017, 6:59 p.m., Hitesh Kh
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61420/#review182179
---
Ship it!
Ship It!
- Brian Rowe
On Aug. 3, 2017, 10:07 p.m
geode-lucene, there is some per-region configuration that
gets added to cache.xml files. Is there any global configuration?
Thanks,
Brian Rowe and Galen O'Sullivan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61540/#review182617
---
Ship it!
Ship It!
- Brian Rowe
On Aug. 9, 2017, 8:45 p.m
nes 62 (patched)
<https://reviews.apache.org/r/61672/#comment259091>
Does this call do anything? serverConnection is a mock and you haven't told
it how to handle this call.
- Brian Rowe
On Aug. 15, 2017, 8:46 p.m., Bruce
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61816/#review183617
---
Ship it!
Ship It!
- Brian Rowe
On Aug. 23, 2017, 3:47 p.m
Currently the default value for this parameter covers the default locator
and server port values. As a result of this, when launching a locator and
then a server on the same system using gfsh, it's possible to see the
server fail because the locator has already bound the default server
socket. We
I agree that we should have defaulted everything to using ephemeral ports
and forced clients to explicitly assign ports and membership ranges if
needed (any of the reasons Anthony mentioned above). However, I don't
think we can change the default assigned ports at this point, unless we
want to try
en ports before selecting ephemeral
> ports? I think this would resolve the collision issue described above.
>
> On Thu, Oct 11, 2018 at 9:58 AM Brian Rowe wrote:
>
> > I agree that we should have defaulted everything to using ephemeral ports
> > and forced clients to explic
e code base? It seems like having a common
> implementation of this would make it a lot easier to solve this type of
> problem in the future.
>
> On Thu, Oct 11, 2018 at 11:27 AM Brian Rowe wrote:
>
> > Galen, this was actually my first instinct. Unfortunately this range is
>
52 matches
Mail list logo
