Re: Review Request 35542: GEODE-51 - Hostname for clients property for cache servers

). - Kirk Lund On June 16, 2015, 11:18 p.m., William Markito wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35542

Re: geode contribution process proposal

+1 Welcome Jun! On Friday, June 12, 2015, jun aoki ja...@apache.org wrote: Hi Geode community My name is Jun Aoki and I work for Pivotal's Ambari project, and am super excited to contribute to Geode! I'd like to propose a few things about contribution process. The Geode community should

Re: geode contribution process proposal

Hi Jun, Here's some info on Geode tests. 1) junit These are unit tests involving mocks as well as end-to-end functional tests. These are currently marked with @Category(UnitTest.class) or @Category(IntegrationTest.class). We need to encourage Geode developers to write a lot of more of the

Re: Review Request 34814: GEODE-38. Gfsh init script ignored

1) System Rules for JUnit 4 I really like System Rules for JUnit 4. This library has been added to our build on GitLabs develop but not in ASF git yet http://stefanbirkner.github.io/system-rules/. This line resets all System Properties changes you make in your tests: @Rule public final

Submitting podling report

According to https://wiki.apache.org/incubator/July2015, the podling report is due today. I don't see where I'm supposed to submit it to though. I'm not seeing any answers on http://incubator.apache.org/guides/mentor.html#Sending+in+an+Incubation+Report either. Any pointers on where to send or

Re: Using the LocatorLauncher

Yeah, the LocatorLauncher is intended to be an API for starting a Locator in your local JVM or to use as the main when starting a new JVM. You don't need to have a gemfire.properties file. You can programmatically define all of your config options at runtime and either feed them in via the

Geode podling report for review

Geode Geode is a data management platform that provides real-time, consistent access to data-intensive applications throughout widely distributed cloud architectures. Geode has been incubating since 2015-04-27. Three most important issues to address in the move towards graduation: 1.

Repackaging src code to org.apache.geode

I just want to clarify that the reason we are planning to repackage is because of this policy: http://incubator.apache.org/guides/mentor.html#repackaging ...which states that repackaging is recommended, but not mandated. Is there a different source that states that it's mandated? -Kirk

Re: Review Request 36221: GEODE-81 - Fixing unit test failures from AnalyzeSerializablesJUnitTest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36221/#review90591 --- Ship it! Ship It! - Kirk Lund On July 6, 2015, 9:55 p.m., Dan

Re: Geode podling report for review

. On Wed, Jul 1, 2015 at 4:20 PM, Kirk Lund kl...@pivotal.io wrote: Geode Geode is a data management platform that provides real-time, consistent access to data-intensive applications throughout widely distributed cloud architectures. Geode has been

Re: Repackaging src code to org.apache.geode

Yep, having 99% of the code in org.apache.geode pkgs with 1% in com.gemstone.gemfire pkgs just to facilitate rolling upgrades seems like something that would be reasonable to discuss on general@incubator. On Thu, Jul 2, 2015 at 12:35 PM, Bruce Schuchardt bschucha...@pivotal.io wrote: Hey, if

Re: Where to place Spark + GemFire connector.

I would think that github would be a better option for the Spark Geode Connector. That way it's not tightly coupled to the Geode release cycle. I don't see why it's desirable to bloat Geode with every single script, tool, or connector that might interact with Geode. Another reason to consider

Re: Review Request 37400: GEODE-211: shorten the time ConcurrentRegionOperationsJUnitTest runs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37400/#review95197 --- Ship it! Ship It! - Kirk Lund On Aug. 12, 2015, 5:16 p.m

Re: Reviewboard seems to be malfunctioning

Check here to see if ASF already knows about it: http://status.apache.org/ If the problem isn't shown on that status page then report it in the ASF infra hipchat: http://infra.chat/ -Kirk On Wed, Aug 19, 2015 at 9:49 AM, Bruce Schuchardt bschucha...@pivotal.io wrote: I get the same error

Re: Investigate test failures: Build 189

I don't have any ideas about the cause without looking at the full stack of RegionDestroyedException and the rest of the logs. We haven't been seeing anything like this so I would expect it to be a real bug that was committed. It might be caused by race condition(s) in either the test or the

Pull requests, wiki page and multiple git commits

bbc2a5fa5f0bbcd00a8ab64451a2528a22d88bc0 Merge: 9efe74e 581702a Author: Kirk Lund kl...@pivotal.io Date: Wed Aug 19 16:09:21 2015 -0700 Merge branch 'feature/GEODE-213' into develop commit 581702ac1d2a9debe8619680d09fa868b9ca8b05 Author: Kirk Lund kl...@pivotal.io Date: Wed Aug 19 16:01:26 2015 -0700

Review Request 37683: Clean up WanCommandsControllerJUnitTest

- gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/web/controllers/WanCommandsControllerJUnitTest.java fb39d8c Diff: https://reviews.apache.org/r/37683/diff/ Testing --- WanCommandsControllerJUnitTest test Thanks, Kirk Lund

Re: There was no nightly build yesterday?

Thanks Mark! On Fri, Aug 21, 2015 at 1:13 PM, Mark Bretl mbr...@pivotal.io wrote: I will re-enable the nightly schedule. --Mark On Fri, Aug 21, 2015 at 1:11 PM, Kirk Lund kirk.l...@gmail.com wrote: Hi, We need to have the nightly build reenabled for Apache Geode incubating. Who do

Re: Review Request 37670: Convert resource string to URI string for log4j2-cli.xml

/logging/LoggingUnitTestSuite.java cd39a9d Diff: https://reviews.apache.org/r/37670/diff/ Testing (updated) --- manual testing with gemfire-assembly/build/install/apache-geode/bin/gfsh LoggingUnitTestSuite LoggingIntegrationTestSuite Thanks, Kirk Lund

Re: Updating JIRA issues

GEODE-47 has some comments but is still open (that's the correct state for now). The others appear to be mistakenly left open after committing fixes. -Kirk On Thu, Aug 20, 2015 at 8:49 PM, Anthony Baker aba...@pivotal.io wrote: It looks like several JIRA issues have been fixed but are still

Re: Review Request 36866: GEODE-127: Improve reliability and increase execution speed of some tests

-mail. To reply, visit: https://reviews.apache.org/r/36866/#review93375 --- On July 28, 2015, 9:27 p.m., Kirk Lund wrote: --- This is an automatically generated e-mail. To reply, visit

Review Request 36957: GEODE-181: Add new FastLogger tests and fix isDebug/TraceEnabled checks

/diff/ Testing --- test, integrationTest, new TestSuites, rebuild open+closed Thanks, Kirk Lund

Re: Review Request 36950: [GEODE-177] Adding Dockerfile to Geode codebase

/#comment148074 Is it possible to use $JAVA_HOME here instead of hardcoding references to /jdk1.8.0_51? - Kirk Lund On July 30, 2015, 8:52 p.m., William Markito wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 36857: Remove rerunTest task from gemfire-core/build.gradle

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36857/#review93301 --- Ship it! Ship It! - Kirk Lund On July 27, 2015, 8:27 p.m., Mark

Re: Review Request 36866: GEODE-127: Improve reliability and increase execution speed of some tests

://reviews.apache.org/r/36866/diff/ Testing --- test, integrationTest and targeted testing of TestSuites Thanks, Kirk Lund

Re: Review Request 37143: fix races in testRegionIdleInvalidate

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37143/#review94316 --- Ship it! Ship It! - Kirk Lund On Aug. 5, 2015, 9:22 p.m

Re: Review Request 36950: [GEODE-177] Adding Dockerfile to Geode codebase

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36950/#review93916 --- Ship it! Ship It! - Kirk Lund On July 31, 2015, 12:13 a.m

Re: Review Request 36957: GEODE-181: Add new FastLogger tests and fix isDebug/TraceEnabled checks

Lund On Aug. 3, 2015, 7:58 p.m., Kirk Lund wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36957/ --- (Updated Aug. 3, 2015

Re: Review Request 36957: GEODE-181: Add new FastLogger tests and fix isDebug/TraceEnabled checks

, Kirk Lund

Re: Review Request 36957: GEODE-181: Add new FastLogger tests and fix isDebug/TraceEnabled checks

: https://reviews.apache.org/r/36957/diff/ Testing --- test, integrationTest, new TestSuites, rebuild open+closed Thanks, Kirk Lund

Re: Review Request 36957: GEODE-181: Add new FastLogger tests and fix isDebug/TraceEnabled checks

TestSuites, rebuild open+closed Thanks, Kirk Lund

Re: Review Request 36963: GEODE-162: remove race condititions and long sleeps from region expiration test methods in RegionTestCase

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36963/#review94131 --- Ship it! Ship It! - Kirk Lund On July 30, 2015, 11:50 p.m

Re: Review Request 37148: fix race in testEntryTtlDestroyEvent

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37148/#review94424 --- Ship it! Ship It! - Kirk Lund On Aug. 5, 2015, 11:52 p.m

Re: 16 days without a green build

For spurious failures, the problematic tests are predominantly dunit tests which are actually end-to-end tests involving 6 jvms. End-to-end tests are notorious for having reliability issues. This is magnified by reusing jvms from one test case to the next. Eventually, one of the 6 jvms hits a

Re: 16 days without a green build

I've created GEODE-218 Change distributedTest task to fork every 1 test case for review and group discussion. -Kirk On Thu, Aug 13, 2015 at 5:29 PM, Kirk Lund kirk.l...@gmail.com wrote: For spurious failures, the problematic tests are predominantly dunit tests which are actually end-to-end

Re: Review Request 37321: GEODE-138: remove race condition from testEventDelivery

? - Kirk Lund On Aug. 10, 2015, 9:28 p.m., Darrel Schneider wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37321

Re: Review Request 37486: GEODE-214: better Azul support

/gemfire/internal/SharedLibrary.java (line 112) https://reviews.apache.org/r/37486/#comment150432 I would recommend fixing the indentation of this method. Also need to make sure the assumption of compressed oops is not used when using Zing JVM. - Kirk Lund On Aug. 14, 2015, 5:22 p.m., Darrel

Re: So is it time to dump Hipchat and move to Geode conversations to Slack?

Did we ever move to Slack? I don't see any further mention of it. It doesn't look like hipchat is being actively used for Geode anymore. -Kirk On Fri, Jul 17, 2015 at 11:26 AM, John Blum jb...@pivotal.io wrote: +1 for moving to Slack On Fri, Jul 17, 2015 at 11:13 AM, Bruce Schuchardt

Re: Review Request 36802: GEODE-86: add getDeserializedValue on OperationContext and add unit test coverage

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36802/#review93133 --- Ship it! Ship It! - Kirk Lund On July 24, 2015, 11:54 p.m

Review Request 36866: GEODE-127: Improve reliability and increase execution speed of some tests

, integrationTest and targeted testing of TestSuites Thanks, Kirk Lund

Re: Review Request 36695: GEODE-148: change expiration tests to not run so long

/gemfire/cache30/RegionTestCase.java (line 3617) https://reviews.apache.org/r/36695/#comment147062 Did you mean to leave the TODO here? - Kirk Lund On July 22, 2015, 4:36 p.m., Darrel Schneider wrote: --- This is an automatically

Re: Review Request 36714: GEODE-143: logging must not cause cached values to be deserialized

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36714/#review92822 --- Ship it! Ship It! - Kirk Lund On July 23, 2015, 12:15 a.m

Re: Review Request 36533: GEODE-128: fix expiration tests to run faster with PR regions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36533/#review92819 --- Ship it! Ship It! - Kirk Lund On July 16, 2015, 12:11 a.m

Re: Review Request 36747: GEODE-152: fix NPE in fetchJvmThreads

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36747/#review92926 --- Ship it! Ship It! - Kirk Lund On July 23, 2015, 9:49 p.m

Re: Review Request 36750: GEODE-153: fix race in testRemoteCacheListener

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36750/#review92956 --- Ship it! Ship It! - Kirk Lund On July 24, 2015, 4:23 p.m

Re: Review Request 36594: Adding try-with-resources for Cache

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36594/#review92925 --- Ship it! Ship It! - Kirk Lund On July 23, 2015, 6:19 a.m

Re: Review Request 36695: GEODE-148: change expiration tests to not run so long

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36695/#review92927 --- Ship it! Ship It! - Kirk Lund On July 22, 2015, 4:36 p.m

Apache Jenkins question

Does anyone know why the Apache Jenkins does not show details on test failures? Our internal Jenkins has a Test Result link with links for each test failure that you can follow to see detailed info on the test. Apache Jenkins doesn't have this and the only way I can see results is in Console

Re: Nightly build test failures

:29 AM, Kirk Lund kl...@pivotal.io wrote: Looks like we have nightly build failures. I think this means we need to freeze all checkins and work to stabilize these test failures: https://builds.apache.org/view/All/job/Geode-nightly/lastFailedBuild/console Jenkins also ran out

Re: git flow feature publish GEODE-nn step fails

the right thing but you are not able to connect to the apache git server. -Dan On Mon, Jul 13, 2015 at 12:01 PM, Kirk Lund kl...@pivotal.io wrote: I had no trouble doing git fetch and starting a feature branch, but now that I'm ready to publish the feature branch and track it from another

git flow feature publish GEODE-nn step fails

2adb1588cce967c2a4149fa674d431871a3066fd Author: Kirk Lund kl...@pivotal.io Date: Mon Jul 13 11:44:21 2015 -0700 GEODE-94: Add typecast to prevent ambiguous method calls under JDK 8 Add typecast to Object for ambiguous logger arguments. *$ git flow feature publish GEODE-94* fatal: unable

Re: Review Request 36415: GEODE-75: System property to Rebalance multiple regions in parallel

That's good enough! Thanks, Kirk On Mon, Jul 13, 2015 at 10:28 AM, Dan Smith dsm...@pivotal.io wrote: On July 13, 2015, 4:30 p.m., Kirk Lund wrote: gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/control/InternalResourceManager.java, line 62 https

Re: Review Request 36416: GEODE-56 Decrementing bytesOverflowedOnDisk when an update happens to an evicted entry.

for this as well as missing categories. - Kirk Lund On July 11, 2015, 12:26 a.m., Dan Smith wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36416

Re: Review Request 36395: GEODE-90: IN SET Query using Index throws IllegalArgumentException

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36395/#review91475 --- Ship it! Ship It! - Kirk Lund On July 10, 2015, 5:11 p.m

Review Request 36454: Typecast ambiguous logger arguments to allow JDK 8 compilation

-jgroups/src/main/java/com/gemstone/org/jgroups/protocols/VIEW_ENFORCER.java 426b8b4 Diff: https://reviews.apache.org/r/36454/diff/ Testing --- Run existing UnitTest, IntegrationTest, DistributedTest suites Thanks, Kirk Lund

Jenkins other testing failures last week

Please join in and select a ticket from the 1st group to assign to yourself. We need to make each of these tests more reliable. Please remove any Thread sleep calls or other actions that can cause a test to intermittently fail. Filed with no assignee yet: GEODE-140

Re: Review Request 39023: GEODE-381 enable gfsh https usage through SSL terminated environments

This is from the old Ant build (which Geode doesn't use): $build.sh run-management-tests -DuseHTTP=true What's the counterpart in the Geode gradle build? -Kirk On Thu, Oct 22, 2015 at 11:55 AM, John Blum wrote: > >

Re: Review Request 39023: GEODE-381 enable gfsh https usage through SSL terminated environments

And I don't > > think there should be - if this is something that needs to be tested it > > should be running in every nightly build, not just when you pass a > special > > flag. > > > > -Dan > > > > On Thu, Oct 22, 2015 at 12:12 PM, Kirk Lund <kl...@pivotal.i

Geode Bug Squashathon Day Three (Wednesday)

On the third day of Geode Bug Squashathon (Wed) we had our 1st GREEN build of DistributedTests this week! We had 11 volunteers fixing 4 tickets (12 more remained in progress by end of day). The UnitTests and IntegrationTests remained green for us except for 1 failure in IntegrationTests. We had

Re: Review Request 39023: GEODE-381 enable gfsh https usage through SSL terminated environments

at 1:53 PM, Anthony Baker <aba...@pivotal.io> wrote: > Can we apply the @Parameterized annotation here? Two code paths, one test > (as the old saying goes…) > > Anthony > > > On Oct 22, 2015, at 1:46 PM, Kirk Lund <kl...@pivotal.io> wrote: > > > &

Re: Review Request 39664: GEODE-487: Remove integrationTest and distributedTest from build task.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39664/#review104108 --- Ship it! Ship It! - Kirk Lund On Oct. 26, 2015, 11:04 p.m

Re: Review Request 39585: Integrated Security DUnitTest

ity/TestAccessControl.java (line 3) <https://reviews.apache.org/r/39585/#comment162464> need to get rid of using hydra.Log gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/security/TestAuthenticator.java (line 3) <https://reviews.apache.org/r/39585/#comment162465&g

Re: Review Request 39659: squash-a-thon week 1 fixes

pache.org/r/39659/#comment162281> indentation inside this block appears to be wrong gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/OnGroupsFunctionExecutionDUnitTest.java (line 188) <https://reviews.apache.org/r/39659/#comment162280> indentation inside thi

Geode Bug Squashathon Day Four

We just finished our 4th day of Geode Bug Squashathon (Thurs) and we've had another GREEN build of DistributedTests. We had 12 volunteers and committed fixes for another 4 tickets. The UnitTest and IntegrationTest suites remained green for us except for 2 failures in IntegrationTests which were

Geode Definition of Done (DOD)

I think we need to settle on a formal Definition of Done (DOD) for Apache Geode. Again, borrowing from Spring Data we could start with what they define and add to it: - Implemented according to specification in the ticket - Unit tested - Integration tested - Reference documentation

JIRA Resolved vs Closed

Is there a standard that ASF follows for when a JIRA ticket changes status to Resolved vs Closed? If not, then I'd like recommend that we follow the process that Spring uses: Ticket changes to Resolved when the fix/implementation is committed or merged (to develop in our case). It then moves to

Re: Review Request 39885: squash-a-thon week 2 fixes

gement/ClientHealthStatsDUnitTest.java (line 418) <https://reviews.apache.org/r/39885/#comment163297> Is this sleep needed? It's happening at the end of the test. - Kirk Lund On Nov. 3, 2015, 1:01 a.m., Darrel Sch

Re: Review Request 39879: Add gemfire-common and @Experimental annotation

the root of the module dependency > tree. > > When we refactor to break dependencies such that we *can* separate out an API > module (both for client and server) we definitely should. > > IMO > > > - Anthony > > On November 2nd, 2015, 9:57 p.m. UTC, Kirk Lund w

Re: Review Request 39346: GEODE-405: sameAs needs to ignore ssl-* properties

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39346/#review105093 --- Ship it! Ship It! - Kirk Lund On Oct. 15, 2015, 5:42 p.m

Re: Review Request 39362: GEODE-406: Adding --J parameter to validate offline-disk-store

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39362/#review105092 --- Ship it! Ship It! - Kirk Lund On Oct. 15, 2015, 11:02 p.m

Re: Review Request 39925: GEODE-438: wait longer in test validation code

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39925/#review105090 --- Ship it! Ship It! - Kirk Lund On Nov. 4, 2015, 12:49 a.m

Re: Review Request 39921: GEODE-296: change test to handle EntryDestroyedException

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39921/#review105091 --- Ship it! Ship It! - Kirk Lund On Nov. 3, 2015, 10:56 p.m

Re: Geode Release 1.0.0

Nitin, The JIRA tickets for CI failures do not have a common parent JIRA, but here's a query to see what's still open: project = GEODE AND status in (Open, "In Progress", Reopened) AND labels = CI There are currently 57 open CI tickets. We are not officially continuing the Squashathon but

Review Request 39879: Add gemfire-common and @Experimental annotation

/build.gradle 1c17474 gemfire-rebalancer/build.gradle d87dd58 gemfire-web/build.gradle f0de25a settings.gradle 4b2da7b Diff: https://reviews.apache.org/r/39879/diff/ Testing --- clean build and unit testing of the new annotation Thanks, Kirk Lund

Re: Review Request 39877: GEODE-522: Fix eclipse classpath for generation of IDE project files

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39877/#review104800 --- Ship it! Ship It! - Kirk Lund On Nov. 2, 2015, 9:38 p.m., Mark

Fwd: Apache Geode Clubhouse: Tuesday, November 3

Reminder: next Clubhouse is tomorrow (Tues) morning 9AM PST! Topic is: OQL Querying and Indexing in Geode Be seeing you, Kirk -- Forwarded message -- From: Marie Nedich Date: Fri, Oct 23, 2015 at 11:26 AM Subject: Apache Geode Clubhouse: Tuesday, November 3

Re: Review Request 39879: Add gemfire-common and @Experimental annotation

g/r/39879/#comment163050> I think this change was added to the diff by latest down-merge of develop HEAD to my branch. Let me know which is correct (testRuntime or provided). - Kirk Lund On Nov. 2, 2015, 9:57 p.m., Kirk Lund

Re: Geode Definition of Done (DOD)

2, 2015 at 12:44 PM, John Blum <jb...@pivotal.io> wrote: > > > >> +1 > >> > >> On Mon, Nov 2, 2015 at 12:36 PM, Udo Kohlmeyer <ukohlme...@pivotal.io> > >> wrote: > >> > >>> +1 > >>> On 3 Nov 2015 7:14 am, &

Re: Geode Definition of Done (DOD)

at 1:33 PM, William Markito <wmark...@pivotal.io> > wrote: > > > > > > +1 > > > > > > Just one question, what is snapshot build deployed ? Is it the same as > > > snapshot build successfully completed ? > > > > > > On Mon, No

Re: testDefaults in BasicI18nJUnitTest

+1 for deleting it. Test looks worthless. I'd be in favor of completely replacing the internationalization code in Geode. -Kirk On Thu, Nov 5, 2015 at 11:43 AM, Dan Smith wrote: > I don't see much purpose behind that test either. I'd be in favor of > deleting it. > > -Dan

Re: Delete feature/GEODE-409?

+1 On Thu, Nov 5, 2015 at 12:19 PM, Sai Boorlagadda wrote: > +1 > > On Thu, Nov 5, 2015 at 12:17 PM, William Markito > wrote: > > > +1 > > > > On Thu, Nov 5, 2015 at 11:39 AM, Darrel Schneider > > > wrote: > > > > > +1 > >

regex in jira

Does anyone know if it's possible to use regex in jira queries/filters? -Kirk

Geode Bug Squashathon Day One

We started off our week of Geode Bug Squashathon with 6 volunteers fixing 12 tickets (with 4 more in-progress). The UnitTests and IntegrationTests are predominantly green. We had a number of failures caused by git timeouts and we hit 2 intermittent failures in IntegrationTests for which we filed

Re: CI cleanup & squashaton

Here's a filter that shows all CI tickets for Geode that are Open, In-Progress or Reopened (excluding HDFS): project = GEODE AND status in (Open, "In Progress", Reopened) AND labels = CI AND labels != HDFS The HDFS tests are disabled and HDFS should probably be moved to a feature branch (ie,

Re: CI cleanup & squashaton

, Oct 16, 2015 at 4:25 PM, Anthony Baker <aba...@pivotal.io> wrote: > +1 > > > On Oct 16, 2015, at 3:33 PM, Kirk Lund <kl...@pivotal.io> wrote: > > > > Here's a filter that shows all CI tickets for Geode that are Open, > > In-Progress or Reopened (excluding

Re: Review Request 39028: fix intermittent failures in TXExpiryJUnitTest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39028/#review101671 --- Ship it! Ship It! - Kirk Lund On Oct. 5, 2015, 11:18 p.m

Re: Review Request 38866: PR put messages no longer deserialize old values early

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38866/#review101670 --- Ship it! Ship It! - Kirk Lund On Sept. 29, 2015, 10:47 p.m

Re: Review Request 39058: GEODE-385: fix test race

/TXExpiryJUnitTest.java (line 324) <https://reviews.apache.org/r/39058/#comment159130> I think I would move this if-else block inside the try { below - Kirk Lund On Oct. 6, 2015, 6:28 p.m., Darrel Schneider wrote: > > --- > This is a

Re: Review Request 39028: fix intermittent failures in TXExpiryJUnitTest

/TXExpiryJUnitTest.java (line 57) <https://reviews.apache.org/r/39028/#comment159126> I would keep this as an IntegrationTest - Kirk Lund On Oct. 5, 2015, 11:18 p.m., Darrel Schneider wrote: > > --- > This is an automatically generated e

Review Request 39290: Configure Log4J 2 on classpath instead of with system property

ne/gemfire/internal/logging/log4j/FastLoggerWithDefaultConfigJUnitTest.java 2e08500 Diff: https://reviews.apache.org/r/39290/diff/ Testing --- precheckin-all Thanks, Kirk Lund

Review Request 36377: Convert FastLoggerJUnitTest to use junit rules

/ Testing --- Unit tests: ./gradlew clean test Thanks, Kirk Lund

Re: Review Request 37737: GEODE-275: fix race in testRegionExpirationAfterMutate

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37737/#review96435 --- Ship it! Ship It! - Kirk Lund On Aug. 24, 2015, 11:23 p.m

Re: Review Request 37702: GEODE-264: Fix varargs compiler warnings

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37702/#review97333 --- Ship it! Ship It! - Kirk Lund On Aug. 22, 2015, 4:03 p.m

Re: Review Request 38015: GEODE-295 - Expecting a suspect string in BridgeMembershipDUnitTet

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38015/#review97332 --- Ship it! Ship It! - Kirk Lund On Sept. 1, 2015, 8:36 p.m., Dan

Re: DRAFT of August report

+1 Looks good Swapnil! On Mon, Aug 31, 2015 at 11:48 PM, Swapnil Bawaskar wrote: > Hi All, > The following is a draft of the August report. Please provide any feedback > you may have. > > Geode: > Geode is a data management platform that provides real-time, consistent >

Re: GEODE-218: Dunit Fork Every 1

+1 I like the idea of switching to fork=1 for a few months to focus on stabilizing any dunit tests that fail without potential test pollution causes. These failures are mostly like ones that involve race conditions. Once we fix these, then we could change back to fork=30. On Fri, Sep 4, 2015 at

Review Request 38133: Move AgentUtilJUnitTest into a package

0f7563b gemfire-assembly/src/test/java/com/gemstone/gemfire/management/internal/AgentUtilJUnitTest.java PRE-CREATION Diff: https://reviews.apache.org/r/38133/diff/ Testing --- integrationTest (AgentUtilJUnitTest is an IntegrationTest) Thanks, Kirk Lund

Re: Review Request 38029: GEODE-306: support printing of off-heap compressed nodes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38029/#review97796 --- Ship it! Ship It! - Kirk Lund On Sept. 1, 2015, 11:46 p.m

Re: Review Request 38135: make pdx logging info level

eople won't run in production with debug or trace. - Kirk Lund On Sept. 4, 2015, 8:50 p.m., Darrel Schneider wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https:

Re: Review Request 38133: Move AgentUtilJUnitTest into a package

> On None, Kirk Lund wrote: > > Ship It! Ignore this. I meant to hit "Publish" instead of "Ship It!" (long week!) - Kirk --- This is an automatically generated e-mail. To reply, visit: https://reviews.ap

  1   2   3   4   5   6   >