Re: Review Request 56801: GEODE-2457: Replace org.apache.geode.internal.FileUtil with org.apache.commons.io.FileUtils

e code. - Ken Howe On Feb. 17, 2017, 10:21 p.m., Kevin Duling wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 56801: GEODE-2457: Replace org.apache.geode.internal.FileUtil with org.apache.commons.io.FileUtils

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56801/#review166005 --- Ship it! Ship It! - Ken Howe On Feb. 17, 2017, 11:56 p.m

Re: Review Request 57050: GEODE-2461: remove unnecessary explicit dependencies

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57050/#review166763 --- Ship it! Ship It! - Ken Howe On Feb. 24, 2017, 9:37 p.m

Re: Review Request 56668: GEODE-2474: mark NetstatDUnitTest as flaky

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56668/#review165545 --- Ship it! Ship It! - Ken Howe On Feb. 14, 2017, 5:01 p.m

Re: Review Request 56637: refactor ServerStarterRule and LocatorStarterRule so that they can be created without a Properties first.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56637/#review165559 --- Ship it! Ship It! - Ken Howe On Feb. 14, 2017, 7 p.m

Re: Review Request 56522: GEODE-2198: do not import cluster config if any region exists to prevent different servers having conflicing region definitions at one point of time.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56522/#review165136 --- Ship it! Ship It! - Ken Howe On Feb. 10, 2017, 12:57 a.m

Re: Review Request 56506: GEODE-2398: Retry oplog channel.write on silent failures

d e-mail. To reply, visit: https://reviews.apache.org/r/56506/#review165186 ------- On Feb. 11, 2017, midnight, Ken Howe wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/565

Re: Review Request 56506: GEODE-2398: Retry oplog channel.write on silent failures

/cache/Oplog.java 0b98364 geode-core/src/test/java/org/apache/geode/internal/cache/OplogFlushTest.java PRE-CREATION Diff: https://reviews.apache.org/r/56506/diff/ Testing --- Started precheckin Thanks, Ken Howe

Re: Review Request 56506: GEODE-2398: Retry oplog channel.write on silent failures

- Ken ------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56506/#review164991 --- On Feb. 11, 2017, midnight, Ken Howe wrote: > >

Re: Review Request 56637: refactor ServerStarterRule and LocatorStarterRule so that they can be created without a Properties first.

/ServerStarterRule.java (lines 89 - 94) <https://reviews.apache.org/r/56637/#comment237407> Looks like a null properties argument will cause an NPE - Ken Howe On Feb. 14, 2017, 4:19 p.m., Jinmei Liao

Review Request 56506: GEODE-2398: Retry oplog channel.write on silent failures

/internal/cache/OplogFlushTest.java PRE-CREATION Diff: https://reviews.apache.org/r/56506/diff/ Testing --- Started precheckin Thanks, Ken Howe

Re: Review Request 56516: GEODE-2456: change JUnit4CacheTestCase tearDown to use deleteQuietly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56516/#review165128 --- Ship it! Ship It! - Ken Howe On Feb. 9, 2017, 8:06 p.m

Re: Build failed in Jenkins: Geode-nightly #762

I’m working on the NetstatDUnitTest failure. GEODE-2541 Ken > On Feb 28, 2017, at 8:35 AM, Apache Jenkins Server > wrote: > > 2: Task failed with an exception. > --- > * What went wrong: > Execution failed for task ':geode-core:flakyTest'. >> There were

Re: Review Request 57221: GEODE-2541: Fixed ClassCastException failure

://reviews.apache.org/r/57221/diff/1-2/ Testing --- Ran spotlessApply Precheckin is in progress Thanks, Ken Howe

Review Request 58080: GEODE-2420: Add classes to estimated the size of exported logs

--- Precheckin is running Thanks, Ken Howe

Re: Review Request 58050: GEODE-2725: export logs --dir now honored when not connected via HTTP[S].

> > (Updated March 30, 2017, 6:17 p.m.) > > > Review request for geode, Jinmei Liao, Jared Stewart, Kevin Duling, Ken Howe, > Kirk Lund, and Swapnil Bawaskar. > > > Repository: geode > > > Description > --- > > exp

Re: Review Request 58050: GEODE-2725: export logs --dir now honored when not connected via HTTP[S].

ated e-mail. To reply, visit: > https://reviews.apache.org/r/58050/ > --- > > (Updated March 30, 2017, 6:17 p.m.) > > > Review request for geode, Jinmei Liao, Jared Stewart, Kevin Duling, Ken Howe, > Kirk Lund, and Swapnil Bawaskar. > > > Repository: geode >

Re: Review Request 58080: GEODE-2420: Add classes to estimated the size of exported logs

216) at org.apache.geode.management.internal.cli.functions.SizeExportLogsFunctionCacheTest.withFunctionError_shouldThrow(SizeExportLogsFunctionCacheTest.java:146) Thanks, Ken Howe

Re: Review Request 57729: GEODE-2634: use log4j levels for auto-completion

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57729/#review169799 --- - Ken Howe On March 17, 2017, 6:48 p.m., Jinmei Liao wrote

Re: Review Request 57729: GEODE-2634: use log4j levels for auto-completion

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57729/#review169795 --- - Ken Howe On March 17, 2017, 6:48 p.m., Jinmei Liao wrote

Re: Review Request 58010: GEODE-2716: export logs default behavior changed from filtering at log level INFO to ALL.

the change noted above.) - Ken Howe On March 28, 2017, 10:34 p.m., Patrick Rhomberg wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 58080: GEODE-2420: Add classes to estimated the size of exported logs

216) at org.apache.geode.management.internal.cli.functions.SizeExportLogsFunctionCacheTest.withFunctionError_shouldThrow(SizeExportLogsFunctionCacheTest.java:146) Thanks, Ken Howe

Re: Review Request 57729: GEODE-2634: use log4j levels for auto-completion

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57729/#review169524 --- Ship it! Ship It! - Ken Howe On March 17, 2017, 6:48 p.m

Re: Review Request 57729: GEODE-2634: use log4j levels for auto-completion

apping is: LogWriter.FINE and FINER --> Level.DEBUG LogWriter.FINEST --> Level.TRACE The new mapping has LogWriter.FINE --> Level.DEBUG LogWriter.FIBNER abnd FINEST --> Level.TRACE - Ken Howe On March 17, 2017, 6:48 p.m.,

Re: Review Request 57573: GEODE-2648: always move deleted files to dirOfDeletedFiles

move the deleted files dir from the dir.listFiles() rather than arbitarily returning length - 1. Something like List listOfFiles = Arrays.asList(dir.listFiles()); listOfFiles.remove(dirOfDeletedFiles.getName()); return listOfFiles.size(); - Ken Howe On March 13, 2017,

Re: Review Request 57573: GEODE-2648: always move deleted files to dirOfDeletedFiles

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57573/#review169054 --- Ship it! Ship It! - Ken Howe On March 15, 2017, 8:02 p.m

Re: Review Request 58080: GEODE-2420: Add classes to estimated the size of exported logs

206) at org.apache.geode.cache.CacheFactory.create(CacheFactory.java:216) at org.apache.geode.management.internal.cli.functions.SizeExportLogsFunctionCacheTest.withFunctionError_shouldThrow(SizeExportLogsFunctionCacheTest.java:146) Re-running precheckin after merging current develop to my branch. Thanks, Ken Howe

Re: Review Request 58319: GEODE-2765: change gfsh help to use log4j levels

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58319/#review171491 --- Ship it! Ship It! - Ken Howe On April 10, 2017, 9:06 p.m

Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

://reviews.apache.org/r/58325/diff/1/ Testing --- All Pulse UI tests run locally pass. Precheckin is running Thanks, Ken Howe

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

y pass. Precheckin is running Thanks, Ken Howe

Re: Review Request 58080: GEODE-2420: Add classes to estimated the size of exported logs

216) at org.apache.geode.management.internal.cli.functions.SizeExportLogsFunctionCacheTest.withFunctionError_shouldThrow(SizeExportLogsFunctionCacheTest.java:146) Re-running precheckin after merging current develop to my branch. Thanks, Ken Howe

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

ing Thanks, Ken Howe

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

/diff/2-3/ Testing (updated) --- ==> Precheckin was clean other than spotlessCheck. Re-runing the precheckin after reformatting to correct the failure All Pulse UI tests run locally pass. Precheckin is running Thanks, Ken Howe

Re: Review Request 58388: GEODE-2730: refactor rules

to have been somewhat haphazard up to now. This commnent applies to all our rules, not just this one. - Ken Howe On April 12, 2017, 3:08 p.m., Jinmei Liao wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 58388: GEODE-2730: refactor rules

ts a boolean to start (or not) later in the before() method. My interpretation is that it's an "auto start" flag for each test. - Ken Howe On April 12, 2017, 3:08 p.m., Jinmei Liao wrote: > > --- > This is an automatic

Re: Review Request 58388: GEODE-2730: refactor rules

> On April 12, 2017, 9:10 p.m., Ken Howe wrote: > > geode-core/src/test/java/org/apache/geode/test/dunit/rules/MemberStarterRule.java > > Lines 110 (patched) > > <https://reviews.apache.org/r/58388/diff/2/?file=1690839#file1690839line110> > > > > I

Re: Review Request 58460: GEODE-2632: refactoring part 1

st/java/org/apache/geode/internal/cache/control/MemoryMonitorJUnitTest.java Lines 297-301 (original), 297-301 (patched) <https://reviews.apache.org/r/58460/#comment245101> Spotless seems to have made a mess of this and the following comment. - Ken Howe On April 14, 2017, 9:

Re: Review Request 58595: GEODE-2681: refactoring to prevent synchronization hang on getAnyInstance

798 (which is a dupilcate of GEODE-2681.) Precheckin has been started. Thanks, Ken Howe

Re: Review Request 58435: GEODE-2647: remove unnecessary client puts from ClientHealthStatsDUnitTest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58435/#review171956 --- Ship it! Ship It! - Ken Howe On April 13, 2017, 10:15 p.m

Re: Review Request 58388: GEODE-2730: refactor rules

ithAutoStart() ? - Ken Howe On April 12, 2017, 11:32 p.m., Jinmei Liao wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://re

Re: Review Request 58518: GEODE-2795: Clean up DUnit VMs after dynamically changing 'user.dir'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58518/#review172364 --- Ship it! Ship It! - Ken Howe On April 18, 2017, 9:58 p.m

Re: Review Request 58518: GEODE-2795: Clean up DUnit VMs after dynamically changing 'user.dir'

/LocatorServerStartupRule.java Line 67 (original), 63 (patched) <https://reviews.apache.org/r/58518/#comment245592> Javadoc comment is no longer needed - Ken Howe On April 19, 2017, 11:22 p.m., Jared Stewart

Re: Review Request 58460: GEODE-2632: refactoring part 1

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58460/#review172121 --- Ship it! Ship It! - Ken Howe On April 17, 2017, 7:09 p.m

Re: Review Request 57390: GEODE-2539: Upgrading Jetty causes RestSecurityItegrationTest to fail

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57390/#review168474 --- Ship it! Ship It! - Ken Howe On March 7, 2017, 7:09 p.m

Re: Review Request 57439: GEODE-2576: delete the zip file on locator after it's being streamed to client.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57439/#review168468 --- Ship it! Ship It! - Ken Howe On March 9, 2017, 12:44 a.m

Re: Review Request 57243: GEODE-2416: Collect together artifacts from individual servers into a single zip file

to the latest (version 3) without examining the intermediate changes. As these changes appear that they won't break nightly builds (your precheckin passed), it would be great to get this merged so that further refinements to the export logs command behavior can be dealt with individually. - Ken Howe

Re: Review Request 57249: GEODE-2488: Remove lsof from netstat command tests

Diff: https://reviews.apache.org/r/57249/diff/2/ Changes: https://reviews.apache.org/r/57249/diff/1-2/ Testing --- Tested locally on JVMs with small heap to verify there are no OOMEs Precheckin has been started. Thanks, Ken Howe

Review Request 57522: check for string "nul" in server/locator status message

en started Thanks, Ken Howe

Re: Review Request 57522: check for string "null" in server/locator status message

Ran spotlessApply Re-started precheckin Thanks, Ken Howe

Re: Review Request 57431: GEODE-2633: When turning on fine logging, GEODE logs the keystore password in clear text

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57431/#review168833 --- Ship it! Ship It! - Ken Howe On March 13, 2017, 6:05 p.m

Re: Review Request 57311: GEODE-2593: add port range to AvailablePortHelper to fix testUDPPortRange

ent240005> Is the upper_bound a useable port? If so then the outer loop test should be '<='. Also in the test condition, the upper_bound should be reduced by the count to enusre that i + j doesn't exceed the upper_bound. - Ken Howe On March 4, 2017, 1:57 a.m.,

Re: Review Request 57242: GEODE-2267: Enhance server/locator startup rules to include workingDir

atched) <https://reviews.apache.org/r/57242/#comment239611> Why use locatorPort from 0 to -1? The test in startServer(Properties properties, int locatorPort, boolean pdxPersistent) is for >0 - Ken Howe On March 2, 2017, 3:30 p.m., Ji

Re: Review Request 57175: GEODE-4160: upgrade mortbay-jetty-servlet-api dependency

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57175/#review167511 --- Ship it! Ship It! - Ken Howe On March 1, 2017, 1:39 a.m

Review Request 57221: GEODE-2541: Fixed ClassCastException failure

/ Testing --- Ran spotlessApply Precheckin is in progress Thanks, Ken Howe

Re: Review Request 61671: GEODE-3328: fix testAddGemFirePropertyFileToCommandLineNew on Windows

/internal/cli/commands/GfshCommandJUnitTest.java Line 411 (original), 411 (patched) <https://reviews.apache.org/r/61671/#comment258982> Why was this test renamed? Not really a problem but on the surface looks unneeded. - Ken Howe On Aug. 15, 2017, 7:29 p.m., Kirk Lund

Re: Review Request 61627: GEODE-3437: Fix list and describe region tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61627/#review183037 --- Ship it! Ship It! - Ken Howe On Aug. 14, 2017, 10:40 p.m

Re: Review Request 61599: GEODE-3328: fix a test failure on windows.

independant constructs is always good. I didn't try this out myself on a Windows machine but the fix looks good. - Ken Howe On Aug. 11, 2017, 10:42 p.m., Jinmei Liao wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 60666: geode-3166: remove the uncalled getCredential method

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60666/#review180683 --- Ship it! Ship It! - Ken Howe On July 17, 2017, 3:49 p.m

Re: Review Request 60985: GEODE-3217: Reimplement gfsh query as a single-step command

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60985/#review181069 --- Ship it! Ship It! - Ken Howe On July 20, 2017, 5:54 p.m

Re: Review Request 60985: GEODE-3217: Reimplement gfsh query as a single-step command

<https://reviews.apache.org/r/60985/#comment256480> Seems there's opportunity for more tests in here, for instance, queryWithInvalidRegionName, queryInvalidExceptionThrown, etc. Have you checked coverage, in particular for the new classes QueryCommand, and QueryInterceptor - Ken Ho

Re: Review Request 60977: GEODE-3251: make JMX test rules more robust

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60977/#review180959 --- Ship it! Ship It! - Ken Howe On July 19, 2017, 5:14 p.m

Re: Review Request 60666: geode-3166: remove the uncalled getCredential method

should be marked as @Deprecated for the upcoming release rather than immediately removing it. Removing the @Deprecated annotation on the 3-arg method is appropriate as this is now the preferred method. - Ken Howe On July 5, 2017, 7:47 p.m., Jinmei Liao wrote

Re: Review Request 58751: GEODE-2632: make GemFireCacheImpl.getRegion(String) non-final

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58751/#review173213 --- Ship it! Ship It! - Ken Howe On April 26, 2017, 8:54 p.m

Re: Review Request 58742: GEODE-2632: minor fixes from code review

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58742/#review173224 --- Ship it! Ship It! - Ken Howe On April 26, 2017, 5:17 p.m

Re: Review Request 58849: GEODE-2840: add a DUnit test to test concurrent deploy

larity to the code. Exceptions would be where the name might conflict with a keyword, which is not the case here. - Ken Howe On April 28, 2017, 6:02 p.m., Jinmei Liao wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 58848: GEODE-2632: prevent ClassCastException from ConnectionCountProbe to Identifiable

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58848/#review173384 --- Ship it! Ship It! - Ken Howe On April 28, 2017, 7:40 p.m

Re: Review Request 61409: GEODE-3328: simplify GfshParserRule

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61409/#review182227 --- Ship it! Ship It! - Ken Howe On Aug. 3, 2017, 5:12 p.m

Re: Review Request 61417: GEODE-3328: adding ssl-truststore-type to the config

/ConfigurationProperties.java Lines 691 (patched) <https://reviews.apache.org/r/61417/#comment258115> Is this declaration needed? It doesn't appear to be used anywhere - Ken Howe On Aug. 3, 2017, 9:15 p.m., Jinmei Liao

Re: Review Request 61196: GEODE-3326: Fix intermittent ConcurrentDeployDUnitTest failure

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61196/#review181674 --- Ship it! Ship It! - Ken Howe On July 27, 2017, 10:22 p.m

Re: Review Request 61480: GEODE-3330: user needs CLUSTER:MANAGE:QUERY permission to create a CQ.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61480/#review182443 --- Ship it! Ship It! - Ken Howe On Aug. 8, 2017, 9:10 p.m

Re: Review Request 61480: GEODE-3330: user needs CLUSTER:MANAGE:QUERY permission to create a CQ.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61480/#review182440 --- Ship it! - Ken Howe On Aug. 8, 2017, 9:10 p.m., Jinmei Liao

Re: Review Request 61487: GEODE-3407: fix deadlock between JMX and reconnect

(cache, cacheServer, ...) - Ken Howe On Aug. 8, 2017, 12:19 a.m., Kirk Lund wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Review Request 61426: GEODE-3277: Fix error path constructors of inner State classes of the Launchers

unit/rules/GfshShellConnectionRule.java e7f17ef208a1708f385c7c4041affb70fd309a4c Diff: https://reviews.apache.org/r/61426/diff/1/ Testing --- Precheckin ran green Thanks, Ken Howe

Re: Review Request 61627: GEODE-3437: Fix list and describe region tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61627/#review183042 --- Ship it! Ship It! - Ken Howe On Aug. 16, 2017, 5:24 p.m

Review Request 61701: GEODE-3277: Fix error path constructors of Launcher inner State classess

/apache/geode/test/dunit/rules/GfshShellConnectionRule.java e7f17ef208a1708f385c7c4041affb70fd309a4c Diff: https://reviews.apache.org/r/61701/diff/1/ Testing --- Precheckin is in progress. Thanks, Ken Howe

Re: Review Request 61701: GEODE-3277: Fix error path constructors of Launcher inner State classess

) --- Precheckin from earlier ran green. Re-running precheckin with this additional refactoring. Thanks, Ken Howe

Re: Review Request 61701: GEODE-3277: Fix error path constructors of Launcher inner State classess

mand correctly shows the locator is online when given the correct `--port=...` value. - Ken --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61701/#review183145 --- On Aug. 16, 2017, 9:21 p.m., Ken Howe wrot

Re: Gitbox enables the full GitHub workflow

+1 Yes, let’s make the move > On Aug 22, 2017, at 11:21 AM, Nabarun Nag wrote: > > +1 > > On Tue, Aug 22, 2017 at 11:15 AM Kirk Lund wrote: > >> +1 to move all our repos to gitbox >> >> On Tue, Aug 22, 2017 at 11:08 AM, Jacob Barrett

Re: Review Request 61701: GEODE-3277: Fix error path constructors of Launcher inner State classess

(updated) --- Re-running precheckin Thanks, Ken Howe

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

tps://reviews.apache.org/r/5/#comment246693> more comment formatting - Ken Howe On May 2, 2017, 12:06 a.m., Kirk Lund wrote: > > --- > This is an automat

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

tps://reviews.apache.org/r/5/#comment246693> more comment formatting - Ken Howe On May 2, 2017, 12:06 a.m., Kirk Lund wrote: > > --- > This is an automat

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

tps://reviews.apache.org/r/5/#comment246693> more comment formatting - Ken Howe On May 2, 2017, 12:06 a.m., Kirk Lund wrote: > > --- > This is an automat

Re: Review Request 58682: GEODE-2662: Missing keys cause columns to shift in gfsh table display.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58682/#review173528 --- Ship it! Ship It! - Ken Howe On April 27, 2017, 6:30 p.m

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

che/geode/cache/query/internal/QueryUtils.java Line 1503 (original), 1409 (patched) <https://reviews.apache.org/r/5/#comment246589> For consistency, suggest renaming this method to getConditionedRelationshipIndexResultsExpandedToTopOrCGJLevel (similar to the renamed getConditionedI

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

ode/cache/query/internal/index/AbstractIndex.java Line 1223 (original), 1256 (patched) <https://reviews.apache.org/r/5/#comment246788> typo: remove '//' that are left over from the original comment - Ken Howe On May

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

empty inner class we won't get a stack trace at all. So just throw the new exception and let the default constructor fill in the stack trace and casue. If the intent was to not have the cause filled in then couldn't we just throw a new zero-arg CacheException? - Ken Howe On May 2, 2017, 12:

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

views.apache.org/r/5/#comment246856> typo - correspnds to my note on line 111 - Ken Howe On May 2, 2017, 12:06 a.m., Kirk Lund wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > h

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

/InternalLocator.java Lines 97-98 (original), 96-97 (patched) <https://reviews.apache.org/r/5/#comment246860> In the javadoc, missing a closing '}' and a ';' between the first two statments - Ken Howe On May 2, 2017, 12:06 a.m., Kirk Lund

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

d and cancelled are considered correct, and we use both of them. - Ken Howe On May 3, 2017, 10:10 p.m., Kirk Lund wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https:

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

che.org/r/5/#comment246892> Suggest rewording this comment "For a long time conflict checks were turned off ..." - Ken Howe On May 3, 2017, 10:10 p.m., Kirk Lund wrote: > > --- > This is an automatically ge

Re: Review Request 59246: GEODE-2876: reset isGfshVM flag to false when tearing down tests using CliCommandTestBase.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59246/#review175004 --- Ship it! Ship It! - Ken Howe On May 12, 2017, 10:12 p.m

Review Request 59287: GEODE-2420: Enable export logs size estimation and user warning

nly DistributedTest still running Thanks, Ken Howe

Re: Review Request 59287: GEODE-2420: Enable export logs size estimation and user warning

tps://reviews.apache.org/r/59287/diff/1-2/ Testing --- Precheckin is in progress - all green so far with only DistributedTest still running Thanks, Ken Howe

Re: Review Request 59299: GEODE-2874: Fix StringIndexOutOfBoundsException while initializing logger

che.org/r/59299/#comment248354> I'd favor a more descriptive test name that won't prompt digging through a JIRA to figure out what the intent is. "testGeode2874_nameWithoutExtensionDoesntThrow"? - Ken Howe On May 15, 2017, 10:04 p.m

Re: Review Request 59299: GEODE-2874: Fix StringIndexOutOfBoundsException while initializing logger

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59299/#review175041 --- Ship it! Ship It! - Ken Howe On May 15, 2017, 10:31 p.m

Re: Review Request 59098: GEODE-2876: add logging to diagnose parser test failure in Jenkins

added org.apache.logging.log4j.Logger as an import rather than specifying the full class path in the declaration. - Ken Howe On May 9, 2017, 3:32 p.m., Jinmei Liao wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 59210: GEODE-2912: Hot deploy for functions in deployed Jars

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59210/#review174826 --- Ship it! Ship It! - Ken Howe On May 12, 2017, 6:06 p.m

Re: Review Request 59961: GEODE-3048: Introduce a rule to identify tests that require GEODE_HOME

/dunit/rules/RequiresGeodeHome.java Lines 28 (patched) <https://reviews.apache.org/r/59961/#comment251315> Many of us are in the habit of putting '\n' in message strings, but I think using LINE_SEPARATOR would be better. - Ken Howe On June 9, 2017, 11:35 p.m., Jared Stewart

Re: Review Request 60199: GEODE-3092: fix specifiedDefaultValue for cacheLoader and cacheWriter

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60199/#review178272 --- Ship it! Ship It! - Ken Howe On June 19, 2017, 4:09 p.m

Re: Review Request 60348: GEODE-3103: GfshRule no longer clutters output

ing quotes on the value arg. -OR- change those tests to be consistent with adding the quotes here. - Ken Howe On June 21, 2017, 10:51 p.m., Jared Stewart wrote: > > --- > This is an automatically generated e-mail. To reply, visit:

  1   2   >