clear query| facets| time Search criteria: author:"Owen Nichols".   Results from 1 to 10 from 324 (0.0s).
Loading phrases to help you
refine your search...
[expand - 3 more] - [DISCUSS] reduce PR checks to JDK11 only - Geode - [mail # dev]
...Currently every PR commit triggers both JDK8 and JDK11 versions of each test job.  I propose that we can eliminate the JDK8 version of each check.  In the extremely rare case where...
   Author: Owen Nichols , 2019-05-16, 01:09
[expand - 2 more] - [DISCUSS] is it time to make Windows tests gating? - Geode - [mail # dev]
...For a very long time we’ve had Windows tests in the main pipeline (hidden away, not in the default view), but the pipeline proceeds to publish regardless of whether Windows tests fail or eve...
   Author: Owen Nichols , 2019-05-16, 01:28
[expand - 1 more] - [DISCUSS] run Windows tests concurrently with Linux tests in main pipeline - Geode - [mail # dev]
...Currently we wait to run the Windows tests for a commit until all the Linux tests have passed.  This results in unnecessarily long feedback time to know if a commit passed on Windows.Th...
   Author: Owen Nichols , 2019-05-16, 01:19
Changing external methods to no longer throw UnsupportedOperationException - Geode - [mail # dev]
...+1I see no semantic difference between adding a new method vs implementing a stub that previously threw UnsupportedOperationException> On May 23, 2019, at 12:56 PM, Udo Kohlmeyer  wr...
   Author: Owen Nichols , 2019-05-23, 20:00
[expand - 2 more] - [DISCUSS] Remove exception.getMessage() error handling - Geode - [mail # dev]
...Here’s an example of a message that was logged before Jack’s change:l192.168.99.1: nodename nor servname provided, or not knownHere’s what it will look like now with .toString() instead of ....
   Author: Owen Nichols , 2019-05-26, 02:28
Unnecessary uses of final on local variables - Geode - [mail # dev]
...I recommend:* Use final if you want to* Don’t -1 someone’s PR because they did or didn’t use final.Personally, there is one case where I like to use final on a local variable: forcing the co...
   Author: Owen Nichols , 2019-06-18, 22:23
[expand - 4 more] - [DISCUSS] require reviews before merging a PR - Geode - [mail # dev]
...It seems common for Geode PRs to get merged with only a single green checkmark in GitHub.According to https://www.apache.org/foundation/voting.html we should not be merging PRs with fewer th...
   Author: Owen Nichols , 2019-05-30, 22:51
[expand - 2 more] - [DISCUSS] Criteria for PMC, committers - Geode - [mail # dev]
...A 6-month waiting period from committer to PMC is tempting because it’s easy to implement, but as you described it yourself, it is arbitrary, which ultimately de-values what it means to be a...
   Author: Owen Nichols , 2019-05-30, 23:17
[expand - 1 more] - [DISCUSS] Disable merge for failing pull requests - Geode - [mail # dev]
...I’d like to follow up on this discussion from late last year.  Six months ago, Kirk wrote:> After we get it more consistently GREEN, I would be willing to change my vote to +1.While ...
   Author: Owen Nichols , 2019-06-04, 23:06
[expand - 1 more] - what is the best way to update a geode pull request - Geode - [mail # dev]
...Personally, I do not force-push to my PRs once any review comments have been accumulated, for the reasons you mention.Not sure if some people just force-push out of habit, or if the requirem...
   Author: Owen Nichols , 2019-05-31, 20:28