-quickcheck and looked into the data it
generates: integral numbers seem pretty reasonable. Strings tend to be
short-ish (length up to hundreds with hundreds of iterations, thousands with
thousands), but are made up of random codepoints, which is nice.
Thanks,
Galen O'Sullivan
(length up to hundreds with hundreds of iterations, thousands with
thousands), but are made up of random codepoints, which is nice.
Thanks,
Galen O'Sullivan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56129/#review163680
---
Ship it!
Ship It!
- Galen O'Sullivan
On Jan. 31, 2017, 5:03
t;
> (Updated Jan. 20, 2017, 4:50 p.m.)
>
>
> Review request for geode, Bruce Schuchardt, Galen O'Sullivan, and Hitesh
> Khamesra.
>
>
> Repository: geode
>
>
> Description
> ---
>
> the add fields have the same behavior and identical co
---
>
> (Updated Jan. 20, 2017, 4:50 p.m.)
>
>
> Review request for geode, Bruce Schuchardt, Galen O'Sullivan, and Hitesh
> Khamesra.
>
>
> Repository: geode
>
>
> Description
> ---
>
> the add fields have the same behavior and identica
a
(line 725)
<https://reviews.apache.org/r/55738/#comment235239>
Don't forget to run spotless so that the build passes, even though it will
ruin your formatting.
- Galen O'Sullivan
On Jan. 20, 2017, 4:45 p.m., U
/cache/tier/sockets/AcceptorImplDUnitTest.java
PRE-CREATION
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/AcceptorImplJUnitTest.java
7aa11b7ca
Diff: https://reviews.apache.org/r/55742/diff/
Testing
---
precheckin in progress.
Thanks,
Galen O'Sullivan
ion.
- Galen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55742/#review162521
---
On Jan. 20, 2017, 11:20 p.m., Galen O'Sullivan wrot
nsider a thread that pauses on the `synchronized` block while another is
inside shutting down.)
- Galen O'Sullivan
On Jan. 24, 2017, 9:53 p.m., Galen O'Sullivan wrote:
>
> ---
> This is an automatically generated e-mail. To reply,
/internal/cache/tier/sockets/AcceptorImpl.java
060683de6
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/AcceptorImplDUnitTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/55742/diff/
Testing
---
precheckin started.
Thanks,
Galen O'Sullivan
/TCPConduit.java (line
927)
<https://reviews.apache.org/r/56093/#comment235029>
The `getLocalAddr` / `getLocalAddress` / (and now) `getAddress` mix here is
confusing. Do you understand the code well enough to refactor these into two or
preferably one method?
- Galen O'Sullivan
On Jan. 30, 2
,
Galen O'Sullivan
investigation.
Diffs (updated)
-
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/AcceptorImpl.java
060683de6
Diff: https://reviews.apache.org/r/55742/diff/
Testing
---
precheckin in progress.
Thanks,
Galen O'Sullivan
/cache/tier/sockets/AcceptorImpl.java
060683de6
Diff: https://reviews.apache.org/r/55742/diff/
Testing (updated)
---
precheckin in progress.
Thanks,
Galen O'Sullivan
ll throw
an InterrruptedException.
- Galen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55742/#review162479
---
On J
).
Thanks,
Galen O'Sullivan
il. To reply, visit:
https://reviews.apache.org/r/55742/#review162512
-------
On Jan. 24, 2017, 10 p.m., Galen O'Sullivan wrote:
>
> ---
> This is an automat
/membership/gms/mgr/GMSMembershipManager.java
(line 980)
<https://reviews.apache.org/r/56986/#comment238540>
Is it worth checking for dcReceiver after this? Do you know what the
difference is?
- Galen O'Sullivan
On Feb. 23, 2017, 4:17 p.m., Bruce Schuchardt
code is confusing.
We're waiting at the end of `findServices`, not trying to update services,
and doing a dumb wait? Or do we get interrupted when the services are updated?
- Galen O'Sullivan
On Feb. 23, 2017, 9:50 p.m., Bruc
,
Galen O'Sullivan
/pholser/junit-quickcheck) for these types
of tests.
Thanks,
Galen O'Sullivan
something like
[junit-quickcheck](https://github.com/pholser/junit-quickcheck) for these types
of tests.
Thanks,
Galen O'Sullivan
/java/org/apache/geode/distributed/internal/membership/gms/membership/GMSJoinLeaveJUnitTest.java
05ab6f7e5
Diff: https://reviews.apache.org/r/58155/diff/1/
Testing
---
Running precheckin.
Ran the test suite in IntelliJ a few hundred times on my machine.
Thanks,
Galen O'Sullivan
casting everything; deleting
old comments and commented code; making things more private. I don't know CQs
or Lucene but it doesn't look like you've changed anything in how they work.
Excellent!
- Galen O'Sullivan
On April 19, 2017, 11:09 p.m., Kirk Lund wrote
to the setting and make the
other side more resilient to single missed pings. What is this pinging?
- Galen O'Sullivan
On March 9, 2017, 10:38 p.m., Hitesh Khamesra wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit
enerated e-mail. To reply, visit:
https://reviews.apache.org/r/58155/#review170927
---
On April 3, 2017, 6:31 p.m., Galen O'Sullivan wrote:
>
> ---
> This is an automati
/java/org/apache/geode/cache30/ReconnectWithCacheXMLDUnitTest.java
Lines 93 (patched)
<https://reviews.apache.org/r/58187/#comment243953>
So Cache is a singleton except when it gets replaced by reconnection? Whoa.
- Galen O'Sullivan
On April 4, 201
/
Changes: https://reviews.apache.org/r/58155/diff/1-2/
Testing (updated)
---
Precheckin passed.
Ran the test suite in IntelliJ a few hundred times on my machine.
Thanks,
Galen O'Sullivan
he mock here.
Would a spy be easier to understand?
geode-protobuf/src/test/java/org/apache/geode/protocol/protobuf/operations/PutAllRequestOperationHandlerJUnitTest.java
Lines 68 (patched)
<https://reviews.apache.org/r/60718/#comment255537>
The serialization service is stateless. Would a spy
What's the general opinion of @NotNull annotations? Would they be useful?
On Fri, Jul 14, 2017 at 9:23 AM, John Blum wrote:
> +1 as well.
>
> However, I will caution this... use Java 8's new java.util.Optional class
> in your codebase judiciously. Using it everywhere,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60834/#review180546
---
Ship it!
Ship It!
- Galen O'Sullivan
On July 13, 2017, 12
I have had a lot of tests fail because mcast-port is nonzero. So now I set
that system property in a lot of new tests. Maybe the tests that set it to
something other than the default should be passing a Properties object to
CacheFactory rather than messing with system properties.
I think the
keep builds from failing.
Diffs
-
geode-assembly/src/test/resources/expected_jars.txt 62601677f
Diff: https://reviews.apache.org/r/60550/diff/1/
Testing
---
Built on my machine, verified that the test passes.
Thanks,
Galen O'Sullivan
bility tests, but do we
want to be testing agains current version and testing version or current and
9.0? Will backwards compatibility set `testVersion` to 9.0 among other versions?
- Galen O'Sullivan
On June 30, 2017, 4:33 p.m., Bruce Schu
the base commit?
geode-assembly/build.gradle
Lines 143 (patched)
<https://reviews.apache.org/r/60312/#comment253398>
It's a nit, but it would be nice to remove trailing whitespace.
- Galen O'Sullivan
On June 21, 2017, 9:35 p.m., Hitesh Khamesra
> On June 27, 2017, 6:19 p.m., Galen O'Sullivan wrote:
> > The diff doesn't apply cleanly on develop on my machine. What's the base
> > commit?
Or better, can you merge or rebase on develop please?
- Galen
---
This is a
I just filed GEODE- for GMSJoinLeaveJUnitTest.
testViewNotSentWhenShuttingDown . I've gotten a failure from the command
line, but not when I run it from within IntelliJ (even repeatedly).
On Fri, Jul 28, 2017 at 4:12 PM, Bruce Schuchardt
wrote:
> (GEODE-3155)
> On June 8, 2017, 11:45 p.m., Galen O'Sullivan wrote:
> > This makes sense to me: we remove the locators if we can't connect to them.
> >
> > I wonder what happens if the two locators can't talk to each other (at
> > first, anyways) but can talk to the res
-
>
> (Updated June 8, 2017, 12:20 a.m.)
>
>
> Review request for geode, Alexander Murmann, Bruce Schuchardt, Galen
> O'Sullivan, Hitesh Khamesra, and Brian Rowe.
>
>
> Repository: geode
>
>
> Description
> ---
reviews.apache.org/r/60157/diff/2/?file=1752476#file1752476line58>
> >
> > surely we don't need this here... How do we handle the case if we have
> > another client protocol handler... this would fail.
We can test this at the factory level. I hadn't been thinking of thi
o reply, visit:
https://reviews.apache.org/r/60157/#review178279
---
On June 20, 2017, 12:53 a.m., Galen O'Sullivan wrote:
>
> ---
> This is an automatically generated e-mail. To
equivalent. If you want me to run again, I will.)
Thanks,
Galen O'Sullivan
ws.apache.org/r/60217/#comment252365>
Do you know what this ignore does?
geode-protobuf/src/test/java/org/apache/geode/serialization/registry/CodecRegistryJUnitTest.java
Lines 58 (patched)
<https://reviews.apache.org/r/60217/#comment252367>
This test can have an expected except
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60217/#review178541
---
Ship it!
Ship It!
- Galen O'Sullivan
On June 21, 2017, 7:01
in `excludeClasses.txt` instead
of `sanctionedSerializables.txt`.
Thanks,
Galen O'Sullivan
settings.gradle c0fdb6e4f
Diff: https://reviews.apache.org/r/60394/diff/2/
Changes: https://reviews.apache.org/r/60394/diff/1-2/
Testing
---
Precheckin passed on v1 (with the serializable in `excludeClasses.txt` instead
of `sanctionedSerializables.txt`.
Thanks,
Galen O'Sullivan
gradle/rat.gradle 1bea5843b
settings.gradle c0fdb6e4f
Diff: https://reviews.apache.org/r/60394/diff/2/
Testing
---
Precheckin passed on v1 (with the serializable in `excludeClasses.txt` instead
of `sanctionedSerializables.txt`).
Thanks,
Galen O'Sullivan
, but I'd consider
it pretty much equivalent. If you want me to run again, I will.)
Thanks,
Galen O'Sullivan
, though.
settings.gradle
Line 39 (original), 39 (patched)
<https://reviews.apache.org/r/60217/#comment252339>
`geode-protobuf` will also need to be added to `geode-assembly` before this
is functional, but we can do that in the work that combines this with the
`ServerConnection` wor
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60217/
> ---
>
> (Updated June 21, 2017, 12:02 a.m.)
>
>
> Review request for geode, Alexander Murmann, Bruce Schuchardt, Galen
&g
/geode/internal/cache/ha/ThreadIdentifierJUnitTest.java
Lines 40 (patched)
<https://reviews.apache.org/r/60446/#comment253364>
Is it possible for a Membership ID to be less than 17 bytes and this to be
negative?
- Galen O'Sullivan
On June 26, 2017, 10:24 p.m., Bruce Schu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60442/#review179007
---
Ship it!
Ship It!
- Galen O'Sullivan
On June 26, 2017, 6:31
/59546/#comment249432>
Could this be split into two tests, one of which continues and the other of
which triggers a failure later on?
- Galen O'Sullivan
On May 25, 2017, 5:12 p.m., Hitesh Khamesra wrote:
>
> ---
> This i
> On May 25, 2017, 5:51 p.m., Galen O'Sullivan wrote:
> > geode-core/src/test/java/org/apache/geode/distributed/internal/DistributionConfigJUnitTest.java
> > Lines 346 (patched)
> > <https://reviews.apache.org/r/59546/diff/1/?file=1732164#file1732164line346>
>
> On May 25, 2017, 11:06 p.m., Darrel Schneider wrote:
> > I think in this case it would have been better to just drop GemFire instead
> > of renaming it to Geode.
> > I see no reason why the product name should be included in these messages.
> > The gfsh message can just say "Could not process
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59866/#review177196
---
Ship it!
Ship It!
- Galen O'Sullivan
On June 6, 2017, 10:50
through the tests.
- Galen O'Sullivan
On June 6, 2017, 11:21 p.m., Udo Kohlmeyer wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
, but I'd consider
it pretty much equivalent. If you want me to run again, I will.)
Thanks,
Galen O'Sullivan
We'll move everything to internal soon, before public release.
We have a ticket to fix the Javadocs and the links will be removed soon as
well. The broken links are for generated files that don't seem to have been
created when the javadoc task runs.
On Fri, Sep 22, 2017 at 2:27 PM, Dan Smith
+1 This is great! I'll take a look at your PR when I get the time.
We may want to think carefully about how often we run these tests, because
unlike regular unit tests, they will take forever to run.
On Fri, Sep 15, 2017 at 1:42 PM, Michael William Dodge
wrote:
> +1 for unit
On Thu, Oct 5, 2017 at 3:14 PM, Jinmei Liao wrote:
> Not sure if this is useful to everyone, but when I push a subsequent commit
> to my feature branch, I always use "force push", so that it's only one commit
> I need to rebase to develop.
>
>
I use force push when I've made
Looks like the repos have moved to gitbox. It should be as simple as
changing the three git-wip-us.apache.org links to gitbox.apache.org .
On Tue, Sep 5, 2017 at 5:01 PM, Karen Miller wrote:
> A guarded +1 based on limited testing of the RC:
>
> Built the Geode RC from
think `allow`
would be clearer than `disallow` because it avoids double-negatives. Can we use
`allow-internal-messages-without-credentials` and have it default to `true`?
- Galen O'Sullivan
On Sept. 5, 2017, 5:57 p.m., Bruce Schuchardt wrote
Replies inline.
On Mon, Sep 25, 2017 at 12:13 PM, Udo Kohlmeyer
wrote:
> Replies inline
> On Mon, Sep 25, 2017 at 11:21 AM, Dan Smith wrote:
>
> > This actually brings up another point I was going to ask about. I don't
> see
> > any version information
t;
> > On Tue, Sep 26, 2017 at 10:48 AM, Mark Bretl <mbr...@apache.org> wrote:
> >
> > > The Apache Geode Project Management Committee has invited Galen
> > > O'Sullivan to join the Geode PMC and we are pleased to announce he has
> > > accepted.
>
I'm having trouble running tests in IntelliJ because of errors with
compilation. Does anyone know why?
I've done a clean build and reloaded Gradle configuration in IntelliJ.
Thanks,
Galen
Information:java: Errors occurred while compiling module 'geode-core'
Information:javac 1.8.0_111 was used
Currently, we have a setting for the new client protocol that controls
whether authentication is required or not. We expect to expand this in the
future, and also that there may be more configuration options for the
protocol. We would like to namespace the settings for this protocol but
don't
It looks like it's moved. If you go to the URL in a browser it will give
you a hint. This command should work to update your remote:
git remote set-url
https://gitbox.apache.org/repos/asf/geode.git
running `git remote -v` after should confirm the
549>
This comparison is changed with byte vs int rules. Probably better to use
the int comparison.
Thismight be even better as a method implemented on CommunicationMode.
- Galen O'Sullivan
On Aug. 30, 2017, 8:48 p.m., Bruce Schuchardt
I don't care too much about exactly what the configuration looks like, but
I want it to be unified, and I want it to be set when the cache starts.
Checking system properties throughout the codebase whenever we feel like is
a bit too magic for me.
In addition, it seems that in order to add a new
Hi all,
I'm thinking about the possibility of making a "Geode public API" module,
mostly containing interfaces that Geode implements.
Why would one do this, you may ask? Here are some reasons.
* It makes clear what is our public API (as opposed to "internal" type
packages).
* Users only have to
If you're talking about "Performance is Key. Consistency is a must", it
looks like the font is "Klavika Web Basic Light".
I see the following in the CSS for that element:
font-family: 'klavika-web', 'Open Sans', 'Helvetica Neue', Helvetica,
Arial, sans-serif !important;
On Mon, Nov 27, 2017 at
Can you show me the particular error? Does it persist? As far as I can
remember, we haven't changed that module this week.
On Fri, Nov 3, 2017 at 9:24 AM, Kirk Lund <kl...@apache.org> wrote:
> Precheckin last night failed to compile.
>
> On Fri, Nov 3, 2017 at 9:22 AM, Galen O'Su
It may be that we missed one of the places where a module gets linked in.
Where did you see this error?
On Fri, Nov 3, 2017 at 9:20 AM, Kirk Lund wrote:
> Anyone know why the build is failing in geode-assembly?
>
> * What went wrong:
> A problem occurred evaluating project
fleshed out, so if there are details you can think of that we might have
missed or any other feedback you can give, I would love to hear it!
https://cwiki.apache.org/confluence/display/GEODE/Protobuf+Value+Encoding+Scheme+Proposal
Thanks,
Galen O'Sullivan
Hi Mike,
The attachment doesn't seem to have attached to your email.
Thanks,
Galen
On Wed, May 23, 2018 at 2:29 PM, Michael Stolz wrote:
> Can someone who has sufficient mojo please update the Apache Geode logo on
> geode.apache.org with the attached version with the TM
:
Hi Galen,
I put a bunch of comments inline in your wiki page. One general question -
is the a proposal to change the default value encoding with the protobuf
protocol from JSON to this new format?
-Dan
On Wed, May 16, 2018 at 5:06 PM, Galen O'Sullivan <gosulli...@pivotal.io>
wrote:
Hi all,
YS
[3] https://dist.apache.org/repos/dist/release/geode/KEYS
[4] https://mirror-vm.apache.org/~henkp/checker/faq.html
On Apr 30, 2018, at 10:31 AM, Galen O'Sullivan <gosulli...@pivotal.io> wrote:
-1
I don't see Mike's key in the KEYS file on either rel/v1.6.0.RC1
(5ce726bd7b4f8d2648fd011a807a1bcc624dd
I'm for it. Less noise is a good thing, and I don't think they're likely
to get prioritized anyways. If we close as WONTFIX or similar, we can
always look back for them later if we want.
On 4/26/18 10:39 AM, Anthony Baker wrote:
Thanks Lynn!
As I first step I’d like to focus on issues
lize it & return it?
> What
> > > > if it isn't PDX-serialized? Do we return a byte array?
> > > >
> > > >
> > > > On 1/24/18 12:21 PM, Dan Smith wrote:
> > > > > Is this really just a workaround for the fact that the
> > re
Hi Addison,
What kind of setup do you have that is causing you to have PDX serialized
objects that cannot be deserialized? Do you have classes that are present
on some servers but not others?
This change would break the contract of region.get() , which is that it
returns a value of a type that
I like this API. I think it saves a lot of boilerplate, and a lot of subtle
possible mistakes. I think the new API would be a bit cleaner than the
RegionVersionVectorTest.doesNotHangIfOtherThreadChangedVersion that Kirk
mentions.
I would like to see generics, especially for `public void
Hi all,
I'm wondering what the collective's opinion of assertions is. I notice
there's an org.apache.geode.internal.Assert class, which is used some
places, and that plain old Java assertions are used in other places. Can we
remove one of these and use the other? Should we be including assertions
+1
On Mon, Aug 27, 2018 at 10:06 AM, Kirk Lund wrote:
> Jira ticket as been filed for this:
>
> GEODE-5639: Replace usage of CatchException with AssertJ and remove
> CatchException dependency
>
> On Thu, Aug 23, 2018 at 4:03 PM, Kirk Lund wrote:
>
> > I goofed on #1. We should be using*
Hi all,
I've noticed a pattern in Geode where we wrap a log call in a check at the
same level, such as:
if (logger.isDebugEnabled()) {
logger.debug("cleaning up incompletely started
DistributionManager due to exception", r);
}
Is there any reason to do this? To my mind,
endar());
> > >3. }
> > >
> > >
> > > Log4j2 lets you pass a format string, so you can just do this:
> > >
> > > logger.debug("Logging in user {} with birthday {}", user.getName(),
> > > user.getBirthdayCalendar());
> > >
> > >
> >
Hi all,
I remembered a couple email threads discussing how it was possible to debug
multiple DUnit processes from IntelliJ, but couldn't find a wiki article or
a very complete description, so I wrote a wiki article. It's a little
barebones right now, but there it is. Please edit or comment with
+1 for making the function service not static, and splitting servers from
clients.
Also +1 for Dan's suggestion.
On Wed, Mar 7, 2018 at 2:51 PM, Patrick Rhomberg
wrote:
> I did not know that! And then, yes, onRegion is much better.
>
> On Wed, Mar 7, 2018 at 2:43 PM, Dan
Hopefully this will fix the pipeline failures. Sorry for the inconvenience.
Best,
Galen
Hi all,
I committed some code recently that looks like it broke the CI. However,
based on the failure, I think it will still fail if I revert the commit.
I'm currently testing a small hotfix and will apply it if it passes tests.
Sorry for the inconvenience.
Thanks,
Galen
GEODE-4836
It looks like there are a few JIRA tasks open to deprecate methods on
DistributedSystem, and it doesn't really have much functionality that would
be useful to a user. I propose that we deprecate DistributedSystem itself,
and keep the system management functionality internal. Is there any reason
Hi all,
We're working on a new protocol for Geode, which means wrapping Geode APIs
and exposing them via the new protocol. For performance and to avoid
class/serialization logic, we want to get these objects in serialized form.
This means calling cache.setReadSerializedForCurrentThread(true)
Hi all,
I'm wondering if it would be possible to have a separate list for commit
announcements, CI emails and similar automated messages, and keep this
mailing list for discussion. It would make it easier to filter email, and I
think it might help improve the discussion. I know I've missed
On Tue, Mar 20, 2018 at 9:06 AM, Patrick Rhomberg
wrote:
>
> @Galen, I had thought it already did that.
>
I guess it does. I never knew. Excellent!
+1 to all of the above.
Will Spotless also reorder imports according to our canonical ordering?
On Mon, Mar 19, 2018 at 4:17 PM, Sai Boorlagadda
wrote:
> +1
>
> On Mon, Mar 19, 2018 at 4:03 PM, Kirk Lund wrote:
>
> > +1
> >
> > On Mon, Mar 19, 2018
Yeah, I think I'm sending myself convinced by Swapnil's argument.
How about muting the "nightly build succeeded" email?
On Wed, Mar 21, 2018 at 9:58 AM, Sean Goller wrote:
> Concourse sends mail whenever a job fails.
>
> On Wed, Mar 21, 2018 at 9:49 AM, Swapnil Bawaskar
It looks like there are two classes called CacheElement in the codebase;
am I correct in thinking that you're referring to
org.apache.geode.cache.configuration.CacheElement?
This class doesn't have a Javadoc, so it's a little hard as an outsider
to understand exactly what it is or how it's
+1 to the new test organization.
And I agree with Patrick's explanation of which is the more stable test
category.
On Thu, Jun 28, 2018 at 9:47 AM, Patrick Rhomberg
wrote:
> @Dale
> It seems to me that the Unit / Distributed / etc would be the more stable
> purpose, and as importantly, each
I think we should run some sort of backwards-compatibility tests between
Java 8 and Java 9/11+. We need testing of Geode (both old and current
versions) on older JVMs talking to Geode on newer JVMs. (for example, what
if Java built-in serialization changes in a way that breaks our code
somehow?)
Would it be feasible to reserve chosen 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
1 - 100 of 200 matches
Mail list logo