gateway sender queue

2019-11-04 Thread Mario Ivanac
Hi geode dev,
we have some questions regarding gateway senders and gateway sender queue.


  *   During testing we have observed, different behavior in parallel and 
serial gateway senders. In case we manually stop, than start gateway senders, 
for parallel gateway senders, queue is purged, but for serial gateway senders 
this is not the case. Is this normal behavior or bug?
  *   What happens with the queues when whole cluster is stopped and later 
started (In our tests with persistent queues, the events are kept)?
  *   Could we add extra option in gfsh command  "start gateway sender" that 
allows to control queues reset (for instance --cleanQueues)?

Thanks,
Mario




Re: Lucene upgrade

2019-11-04 Thread Dan Smith
I think the issue probably has to do with doing a rolling upgrade from an
old version of geode (with an old version of lucene) to the new version of
geode.

Geode's lucene integration works by writing the lucene index to a colocated
region. So lucene index data that was generated on one server can be
replicated or rebalanced to other servers.

I think what may be happening is that data written by a geode member with a
newer version is being read by a geode member with an old version. Because
this is a rolling upgrade test, members with multiple versions will be
running as part of the same cluster.

I think to really fix this rolling upgrade issue we would need to somehow
configure the new version of lucene to write data in the old format, at
least until the rolling upgrade is complete. I'm not sure if that is
possible with lucene or not - but perhaps? Another option might be to
freeze writes to the lucene index during the rolling upgrade process.
Lucene indexes are asynchronous, so this wouldn't necessarily require
blocking all puts. But it would require queueing up a lot of updates.

-Dan

On Mon, Nov 4, 2019 at 12:05 AM Mario Kevo  wrote:

> Hi geode dev,
>
> I'm working on upgrade lucene to a newer version. (
> https://issues.apache.org/jira/browse/GEODE-7309)
>
> I followed instruction from
> https://cwiki.apache.org/confluence/display/GEODE/Upgrading+to+Lucene+7.1.0
> Also add some other changes that is needed for lucene 8.2.0.
>
> I found some problems with tests:
>  * geode-
>lucene/src/test/java/org/apache/geode/cache/lucene/internal/distribu
>ted/DistributedScoringJUnitTest.java:
>
>
>  *
> geode-lucene/src/upgradeTest/java/org/apache/geode/cache/lucene/RollingUpgradeQueryReturnsCorrectResultsAfterClientAndServersAreRolledOver.java:
>  *
> geode-lucene/src/upgradeTest/java/org/apache/geode/cache/lucene/RollingUpgradeQueryReturnsCorrectResultAfterTwoLocatorsWithTwoServersAreRolled.java:
>  *
> ./geode-lucene/src/upgradeTest/java/org/apache/geode/cache/lucene/RollingUpgradeQueryReturnsCorrectResultsAfterServersRollOverOnPartitionRegion.java:
>  *
> ./geode-lucene/src/upgradeTest/java/org/apache/geode/cache/lucene/RollingUpgradeQueryReturnsCorrectResultsAfterServersRollOverOnPersistentPartitionRegion.java:
>
>   -> failed due to
> Caused by: org.apache.lucene.index.IndexFormatTooOldException: Format
> version is not supported (resource
> BufferedChecksumIndexInput(segments_1)): 6 (needs to be between 7 and
> 9). This version of Lucene only supports indexes created with release
> 6.0 and later.
> at
> org.apache.lucene.codecs.CodecUtil.checkHeaderNoMagic(CodecUtil.java:21
> 3)
> at
> org.apache.lucene.index.SegmentInfos.readCommit(SegmentInfos.java:305)
> at
> org.apache.lucene.index.SegmentInfos.readCommit(SegmentInfos.java:289)
> at
> org.apache.lucene.index.IndexWriter.(IndexWriter.java:846)
> at
> org.apache.geode.cache.lucene.internal.IndexRepositoryFactory.finishCom
> putingRepository(IndexRepositoryFactory.java:123)
> at
> org.apache.geode.cache.lucene.internal.IndexRepositoryFactory.computeIn
> dexRepository(IndexRepositoryFactory.java:66)
> at
> org.apache.geode.cache.lucene.internal.PartitionedRepositoryManager.com
> puteRepository(PartitionedRepositoryManager.java:151)
> at
> org.apache.geode.cache.lucene.internal.PartitionedRepositoryManager.lam
> bda$computeRepository$1(PartitionedRepositoryManager.java:170)
> ... 16 more
>
>
>  *
> geode-lucene/src/upgradeTest/java/org/apache/geode/cache/lucene/RollingUpgradeQueryReturnsCorrectResultsAfterClientAndServersAreRolledOverAllBucketsCreated.java:
>
>   -> failed with the same exception as previous tests
>
>
> I found this on web
>
> https://stackoverflow.com/questions/47454434/solr-indexing-issue-after-upgrading-from-4-7-to-7-1
> , but not have an idea how to proceed with that.
>
> Does anyone has any idea how to fix it?
>
> BR,
> Mario
>
>
>
>
>


Re: 1.11.0 Release branch has been cut

2019-11-04 Thread Mark Hanson
Argh... Corrected release version below…

> On Nov 4, 2019, at 4:47 PM, Mark Hanson  wrote:
> 
> Hello Geode Dev Community,
> 
> We have created a new release branch for Apache Geode 1.11.0 - 
> “release/1.11.0"
> 
> Please do review and raise any concern with the release branch.
> If no concerns are raised, we will start with the voting for the release 
> candidate soon.
> 
> Regards
> Mark Hanson
> 



1.11.0 Release branch has been cut

2019-11-04 Thread Mark Hanson
Hello Geode Dev Community,

We have created a new release branch for Apache Geode 1.1100 - “release/1.11.0"

Please do review and raise any concern with the release branch.
If no concerns are raised, we will start with the voting for the release 
candidate soon.

Regards
Mark Hanson



Re: ClassNotFoundException: org.apache.geode.logging.internal.log4j.api.LogService

2019-11-04 Thread Kirk Lund
1) nevermind
2) any time you change branches, do yourself a favor and always do a clean
build:
$ ./gradlew clean
$ ./gradlew build installDist -x test


For anyone who's wondering... yes I keep GEODE_HOME configured in my run
configuration in IntelliJ because I've been following ATDD in my daily work
which means I'm writing acceptance tests every day AND you can't run these
tests in IntelliJ without setting GEODE_HOME.

On Mon, Nov 4, 2019 at 1:18 PM Kirk Lund  wrote:

> Looks like the Management service blows up
> with ClassNotFoundException: 
> org.apache.geode.logging.internal.log4j.api.LogService
>
> This seems related to the previous ClassNotFoundException, except that in
> this instance, the LogService which cannot be found is in the correct
> package. Please disregard the previous email about it being in the wrong
> package.
>
> This email shows the correct error and yes it is occurring on develop even
> when GEODE_HOME is empty or not specified and there are no stale .war or
> .jar files locally. I'm convinced there's a bug here and I also see that
> Pivotal is reporting this same error internally against non-dunit tests so
> I'm not the only one seeing this.
>
> The background dunit locator starts up and then attempts to
> startClusterManagementService which fails because for some reason the
> classpath of that background locator is missing geode-logging which is
> where LogService moved to.
>
> Run any dunit test and you should see the following logged by [locator]. I
> specifically ran this:
>
> $ ./gradlew clean
> $ ./gradlew build
> $ ./gradlew geode-cq:distributedTest --tests PutAllCSDUnitTest
>
> Then search through all of the stdout/err from the test to find...
>
> [error 2019/11/04 13:09:31.730 PST 
> tid=0x13] Context initialization failed
> org.springframework.beans.factory.BeanCreationException: Error creating
> bean with name 'managementControllerAdvice' defined in file
> [/Users/klund/dev/gemfire/geode/geode-cq/dunit/locator/GemFire_klund/services/http/0.0.0.0_7070_management_424997f1/webapp/WEB-INF/classes/org/apache/geode/management/internal/rest/controllers/ManagementControllerAdvice.class]:
> Instantiation of bean failed; nested exception is
> java.lang.NoClassDefFoundError:
> org/apache/geode/logging/internal/log4j/api/LogService
> at
> org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateBean(AbstractAutowireCapableBeanFactory.java:1159)
> at
> org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBeanInstance(AbstractAutowireCapableBeanFactory.java:1103)
> at
> org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:511)
> at
> org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:481)
> at
> org.springframework.beans.factory.support.AbstractBeanFactory$1.getObject(AbstractBeanFactory.java:312)
> at
> org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:230)
> at
> org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:308)
> at
> org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:197)
> at
> org.springframework.beans.factory.support.DefaultListableBeanFactory.preInstantiateSingletons(DefaultListableBeanFactory.java:764)
> at
> org.springframework.context.support.AbstractApplicationContext.finishBeanFactoryInitialization(AbstractApplicationContext.java:867)
> at
> org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:542)
> at
> org.springframework.web.servlet.FrameworkServlet.configureAndRefreshWebApplicationContext(FrameworkServlet.java:668)
> at
> org.springframework.web.servlet.FrameworkServlet.createWebApplicationContext(FrameworkServlet.java:634)
> at
> org.springframework.web.servlet.FrameworkServlet.createWebApplicationContext(FrameworkServlet.java:682)
> at
> org.springframework.web.servlet.FrameworkServlet.initWebApplicationContext(FrameworkServlet.java:553)
> at
> org.springframework.web.servlet.FrameworkServlet.initServletBean(FrameworkServlet.java:494)
> at
> org.springframework.web.servlet.HttpServletBean.init(HttpServletBean.java:171)
> at javax.servlet.GenericServlet.init(GenericServlet.java:244)
> at
> org.eclipse.jetty.servlet.ServletHolder.initServlet(ServletHolder.java:599)
> at
> org.eclipse.jetty.servlet.ServletHolder.initialize(ServletHolder.java:425)
> at
> org.eclipse.jetty.servlet.ServletHandler.lambda$initialize$0(ServletHandler.java:751)
> at
> java.util.stream.SortedOps$SizedRefSortingSink.end(SortedOps.java:352)
> at
> 

ClassNotFoundException: org.apache.geode.logging.internal.log4j.api.LogService

2019-11-04 Thread Kirk Lund
Looks like the Management service blows up
with ClassNotFoundException:
org.apache.geode.logging.internal.log4j.api.LogService

This seems related to the previous ClassNotFoundException, except that in
this instance, the LogService which cannot be found is in the correct
package. Please disregard the previous email about it being in the wrong
package.

This email shows the correct error and yes it is occurring on develop even
when GEODE_HOME is empty or not specified and there are no stale .war or
.jar files locally. I'm convinced there's a bug here and I also see that
Pivotal is reporting this same error internally against non-dunit tests so
I'm not the only one seeing this.

The background dunit locator starts up and then attempts to
startClusterManagementService which fails because for some reason the
classpath of that background locator is missing geode-logging which is
where LogService moved to.

Run any dunit test and you should see the following logged by [locator]. I
specifically ran this:

$ ./gradlew clean
$ ./gradlew build
$ ./gradlew geode-cq:distributedTest --tests PutAllCSDUnitTest

Then search through all of the stdout/err from the test to find...

[error 2019/11/04 13:09:31.730 PST 
tid=0x13] Context initialization failed
org.springframework.beans.factory.BeanCreationException: Error creating
bean with name 'managementControllerAdvice' defined in file
[/Users/klund/dev/gemfire/geode/geode-cq/dunit/locator/GemFire_klund/services/http/0.0.0.0_7070_management_424997f1/webapp/WEB-INF/classes/org/apache/geode/management/internal/rest/controllers/ManagementControllerAdvice.class]:
Instantiation of bean failed; nested exception is
java.lang.NoClassDefFoundError:
org/apache/geode/logging/internal/log4j/api/LogService
at
org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateBean(AbstractAutowireCapableBeanFactory.java:1159)
at
org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBeanInstance(AbstractAutowireCapableBeanFactory.java:1103)
at
org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:511)
at
org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:481)
at
org.springframework.beans.factory.support.AbstractBeanFactory$1.getObject(AbstractBeanFactory.java:312)
at
org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:230)
at
org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:308)
at
org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:197)
at
org.springframework.beans.factory.support.DefaultListableBeanFactory.preInstantiateSingletons(DefaultListableBeanFactory.java:764)
at
org.springframework.context.support.AbstractApplicationContext.finishBeanFactoryInitialization(AbstractApplicationContext.java:867)
at
org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:542)
at
org.springframework.web.servlet.FrameworkServlet.configureAndRefreshWebApplicationContext(FrameworkServlet.java:668)
at
org.springframework.web.servlet.FrameworkServlet.createWebApplicationContext(FrameworkServlet.java:634)
at
org.springframework.web.servlet.FrameworkServlet.createWebApplicationContext(FrameworkServlet.java:682)
at
org.springframework.web.servlet.FrameworkServlet.initWebApplicationContext(FrameworkServlet.java:553)
at
org.springframework.web.servlet.FrameworkServlet.initServletBean(FrameworkServlet.java:494)
at
org.springframework.web.servlet.HttpServletBean.init(HttpServletBean.java:171)
at javax.servlet.GenericServlet.init(GenericServlet.java:244)
at
org.eclipse.jetty.servlet.ServletHolder.initServlet(ServletHolder.java:599)
at
org.eclipse.jetty.servlet.ServletHolder.initialize(ServletHolder.java:425)
at
org.eclipse.jetty.servlet.ServletHandler.lambda$initialize$0(ServletHandler.java:751)
at
java.util.stream.SortedOps$SizedRefSortingSink.end(SortedOps.java:352)
at
java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:482)
at
java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
at
java.util.stream.StreamSpliterators$WrappingSpliterator.forEachRemaining(StreamSpliterators.java:312)
at
java.util.stream.Streams$ConcatSpliterator.forEachRemaining(Streams.java:743)
at
java.util.stream.Streams$ConcatSpliterator.forEachRemaining(Streams.java:742)
at
java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:580)
at
org.eclipse.jetty.servlet.ServletHandler.initialize(ServletHandler.java:744)
at

Passed: apache/geode-native#2176 (release/1.11.0 - 24c5c78)

2019-11-04 Thread Travis CI
Build Update for apache/geode-native
-

Build: #2176
Status: Passed

Duration: 1 hr, 37 mins, and 17 secs
Commit: 24c5c78 (release/1.11.0)
Author: M. Oleske
Message: Resolve LGTM issues in ThinClientPoolDM (#543)

* LGTM issues in ThinClientPoolDM, among other things

* Clean up some warnings from CLion analysis

* Remove unused functions

* Use do-while instead of removing `;`

* Use acquire and block instead of while loop and tryacquire

* Use unique pointer for resultProcessor 

* Add override to be more clear

* Don't check for error twice

Authored-by: Michael Oleske 

View the changeset: 
https://github.com/apache/geode-native/compare/release/1.11.0

View the full build log and details: 
https://travis-ci.org/apache/geode-native/builds/607268389?utm_medium=notification_source=email

--

You can unsubscribe from build emails from the apache/geode-native repository 
going to 
https://travis-ci.org/account/preferences/unsubscribe?repository=11948127_medium=notification_source=email.
Or unsubscribe from *all* email updating your settings at 
https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notification_source=email.
Or configure specific recipients for build notifications in your .travis.yml 
file. See https://docs.travis-ci.com/user/notifications.



Re: ***UNCHECKED*** Re: ClassNotFoundException: LogService

2019-11-04 Thread Dan Smith
Kirk and I tracked this down. He had a GEODE_HOME environment variable
which the locator was picking up and finding an old war file from.

-Dan

On Fri, Nov 1, 2019 at 4:16 PM Dan Smith  wrote:

> Hi Kirk,
>
> Did you do a ./gradlew clean? It looks like you might be picking up an old
> version of one of the geode war files that is referring to the old location
> of the LogService class.
>
> -Dan
>
> On Fri, Nov 1, 2019 at 3:28 PM Kirk Lund  wrote:
>
> > To reproduce, drop this new test in geode-dunit/src/distributedTest/java
> > and then run it in IntelliJ or however you want.
> >
> > It passes, but if you search through the output you'll find the error in
> my
> > first message with:
> >
> > Caused by: java.lang.ClassNotFoundException:
> > *org.apache.geode.internal.logging.LogService*
> > at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
> > at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
> > at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:349)
> > at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> > at
> >
> >
> org.eclipse.jetty.webapp.WebAppClassLoader.loadClass(WebAppClassLoader.java:543)
> > at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> > ... 88 more
> >
> > /*
> >  * Licensed to the Apache Software Foundation (ASF) under one or more
> > contributor license
> >  * agreements. See the NOTICE file distributed with this work for
> > additional information regarding
> >  * copyright ownership. The ASF licenses this file to You under the
> Apache
> > License, Version 2.0 (the
> >  * "License"); you may not use this file except in compliance with the
> > License. You may obtain a
> >  * copy of the License at
> >  *
> >  * http://www.apache.org/licenses/LICENSE-2.0
> >  *
> >  * Unless required by applicable law or agreed to in writing, software
> > distributed under the License
> >  * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS
> OF
> > ANY KIND, either express
> >  * or implied. See the License for the specific language governing
> > permissions and limitations under
> >  * the License.
> >  */
> > package org.apache.geode.test.dunit.examples;
> >
> > import static
> > org.apache.geode.distributed.ConfigurationProperties.HTTP_SERVICE_PORT;
> > import static
> > org.apache.geode.distributed.ConfigurationProperties.JMX_MANAGER_PORT;
> > import static org.apache.geode.internal.AvailablePort.SOCKET;
> > import static
> > org.apache.geode.internal.AvailablePort.getRandomAvailablePort;
> > import static org.apache.geode.test.dunit.VM.getVM;
> > import static org.assertj.core.api.Assertions.assertThat;
> >
> > import java.io.Serializable;
> >
> > import org.junit.After;
> > import org.junit.Before;
> > import org.junit.Rule;
> > import org.junit.Test;
> >
> > import org.apache.geode.distributed.LocatorLauncher;
> > import org.apache.geode.test.dunit.rules.DistributedRule;
> > import
> >
> org.apache.geode.test.junit.rules.serializable.SerializableTemporaryFolder;
> >
> > public class LocatorLauncherDistributedTest implements Serializable {
> >
> >   private static LocatorLauncher locatorLauncher;
> >
> >   @Rule
> >   public DistributedRule distributedRule = new DistributedRule();
> >
> >   @Rule
> >   public SerializableTemporaryFolder temporaryFolder = new
> > SerializableTemporaryFolder();
> >
> >   @Before
> >   public void setUp() {
> > String name = getClass().getSimpleName();
> > int port = getRandomAvailablePort(SOCKET);
> >
> > getVM(0).invoke(() -> {
> >   locatorLauncher = new LocatorLauncher.Builder()
> >
> > .setWorkingDirectory(temporaryFolder.newFolder(name).getAbsolutePath())
> >   .setMemberName(name)
> >   .setPort(port)
> >   .set(HTTP_SERVICE_PORT, "0")
> >   .set(JMX_MANAGER_PORT, "0")
> >   .build();
> >   locatorLauncher.start();
> > });
> >   }
> >
> >   @After
> >   public void tearDown() {
> > getVM(0).invoke(() -> {
> >   locatorLauncher.stop();
> > });
> >   }
> >
> >   @Test
> >   public void isRunning() {
> > getVM(0).invoke(() -> {
> >   assertThat(locatorLauncher.isRunning()).isTrue();
> > });
> >   }
> > }
> >
> >
> > On Fri, Nov 1, 2019 at 3:02 PM Kirk Lund  wrote:
> >
> > > I'm seeing ClassNotFoundExceptions for FQN of the old package of the
> > > LogService:
> > >
> > > [locator] Caused by: java.lang.ClassNotFoundException:
> > > *org.apache.geode.internal.logging.LogService*
> > > [locator] at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
> > >
> > > The above is interesting because it's the old package, not the current
> > > one. The current package is
> > > *org.apache.geode.logging.internal.log4j.api.LogService*.
> > >
> > > There are two related commits and I'm wondering if something didn't get
> > > completed or merged correctly. Do we need to revert any of these?
> Anyone
> > > working on this?
> > >
> > > commit 

Re: ***UNCHECKED*** Re: ClassNotFoundException: LogService

2019-11-04 Thread Kirk Lund
I filed https://issues.apache.org/jira/browse/GEODE-7403 and I'm currently
trying to:

a) determine if it reproduces in both CLI and IntelliJ (IntelliJ so far
with a clean checkout of develop)
b) determine why an error-level log message logged by the background dunit
locator is not causing a LogConsumer (grep-for-suspects) to fail

Note: I cleaned up the stack trace in GEODE-7403 and it's clear that it's
logged when the background dunit locator tries to start the cluster
management service.

I'll update GEODE-7403 as I learn more.

On Mon, Nov 4, 2019 at 9:39 AM Robert Houghton  wrote:

> Kirk, will you craft a Jira about this for the "Build" component, and
> assign it to me?
>
> On Mon, Nov 4, 2019, 09:30 Kirk Lund  wrote:
>
> > If it is picking up an old version of geode then it's doing so because we
> > use old versions for upgrade/backwards testing. The only old versions of
> > the jars I have are the ones that our build pulls down for the upgrade
> > tests. These old versions do tend confuse intellij so it's possible it's
> a
> > problem only in intellij. Since I use the debugger a lot, I use IntelliJ
> a
> > lot and I believe it's vital we keep IntelliJ working for geode for
> > everyone.
> >
> > On Fri, Nov 1, 2019 at 4:16 PM Dan Smith  wrote:
> >
> > > Hi Kirk,
> > >
> > > Did you do a ./gradlew clean? It looks like you might be picking up an
> > old
> > > version of one of the geode war files that is referring to the old
> > location
> > > of the LogService class.
> > >
> > > -Dan
> > >
> > > On Fri, Nov 1, 2019 at 3:28 PM Kirk Lund  wrote:
> > >
> > > > To reproduce, drop this new test in
> > geode-dunit/src/distributedTest/java
> > > > and then run it in IntelliJ or however you want.
> > > >
> > > > It passes, but if you search through the output you'll find the error
> > in
> > > my
> > > > first message with:
> > > >
> > > > Caused by: java.lang.ClassNotFoundException:
> > > > *org.apache.geode.internal.logging.LogService*
> > > > at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
> > > > at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
> > > > at
> > sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:349)
> > > > at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> > > > at
> > > >
> > > >
> > >
> >
> org.eclipse.jetty.webapp.WebAppClassLoader.loadClass(WebAppClassLoader.java:543)
> > > > at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> > > > ... 88 more
> > > >
> > > > /*
> > > >  * Licensed to the Apache Software Foundation (ASF) under one or more
> > > > contributor license
> > > >  * agreements. See the NOTICE file distributed with this work for
> > > > additional information regarding
> > > >  * copyright ownership. The ASF licenses this file to You under the
> > > Apache
> > > > License, Version 2.0 (the
> > > >  * "License"); you may not use this file except in compliance with
> the
> > > > License. You may obtain a
> > > >  * copy of the License at
> > > >  *
> > > >  * http://www.apache.org/licenses/LICENSE-2.0
> > > >  *
> > > >  * Unless required by applicable law or agreed to in writing,
> software
> > > > distributed under the License
> > > >  * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR
> CONDITIONS
> > > OF
> > > > ANY KIND, either express
> > > >  * or implied. See the License for the specific language governing
> > > > permissions and limitations under
> > > >  * the License.
> > > >  */
> > > > package org.apache.geode.test.dunit.examples;
> > > >
> > > > import static
> > > >
> org.apache.geode.distributed.ConfigurationProperties.HTTP_SERVICE_PORT;
> > > > import static
> > > >
> org.apache.geode.distributed.ConfigurationProperties.JMX_MANAGER_PORT;
> > > > import static org.apache.geode.internal.AvailablePort.SOCKET;
> > > > import static
> > > > org.apache.geode.internal.AvailablePort.getRandomAvailablePort;
> > > > import static org.apache.geode.test.dunit.VM.getVM;
> > > > import static org.assertj.core.api.Assertions.assertThat;
> > > >
> > > > import java.io.Serializable;
> > > >
> > > > import org.junit.After;
> > > > import org.junit.Before;
> > > > import org.junit.Rule;
> > > > import org.junit.Test;
> > > >
> > > > import org.apache.geode.distributed.LocatorLauncher;
> > > > import org.apache.geode.test.dunit.rules.DistributedRule;
> > > > import
> > > >
> > >
> >
> org.apache.geode.test.junit.rules.serializable.SerializableTemporaryFolder;
> > > >
> > > > public class LocatorLauncherDistributedTest implements Serializable {
> > > >
> > > >   private static LocatorLauncher locatorLauncher;
> > > >
> > > >   @Rule
> > > >   public DistributedRule distributedRule = new DistributedRule();
> > > >
> > > >   @Rule
> > > >   public SerializableTemporaryFolder temporaryFolder = new
> > > > SerializableTemporaryFolder();
> > > >
> > > >   @Before
> > > >   public void setUp() {
> > > > String name = getClass().getSimpleName();
> > > > int port = 

Errored: apache/geode-examples#387 (release/1.11.0 - 5a57cfb)

2019-11-04 Thread Travis CI
Build Update for apache/geode-examples
-

Build: #387
Status: Errored

Duration: 1 min and 34 secs
Commit: 5a57cfb (release/1.11.0)
Author: Mark Hanson
Message: Prepping for 1.11.0 release

View the changeset: https://github.com/apache/geode-examples/commit/5a57cfba8c9b

View the full build log and details: 
https://travis-ci.org/apache/geode-examples/builds/607266152?utm_medium=notification_source=email

--

You can unsubscribe from build emails from the apache/geode-examples repository 
going to 
https://travis-ci.org/account/preferences/unsubscribe?repository=11483039_medium=notification_source=email.
Or unsubscribe from *all* email updating your settings at 
https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notification_source=email.
Or configure specific recipients for build notifications in your .travis.yml 
file. See https://docs.travis-ci.com/user/notifications.



Re: ***UNCHECKED*** Re: ClassNotFoundException: LogService

2019-11-04 Thread Robert Houghton
Kirk, will you craft a Jira about this for the "Build" component, and
assign it to me?

On Mon, Nov 4, 2019, 09:30 Kirk Lund  wrote:

> If it is picking up an old version of geode then it's doing so because we
> use old versions for upgrade/backwards testing. The only old versions of
> the jars I have are the ones that our build pulls down for the upgrade
> tests. These old versions do tend confuse intellij so it's possible it's a
> problem only in intellij. Since I use the debugger a lot, I use IntelliJ a
> lot and I believe it's vital we keep IntelliJ working for geode for
> everyone.
>
> On Fri, Nov 1, 2019 at 4:16 PM Dan Smith  wrote:
>
> > Hi Kirk,
> >
> > Did you do a ./gradlew clean? It looks like you might be picking up an
> old
> > version of one of the geode war files that is referring to the old
> location
> > of the LogService class.
> >
> > -Dan
> >
> > On Fri, Nov 1, 2019 at 3:28 PM Kirk Lund  wrote:
> >
> > > To reproduce, drop this new test in
> geode-dunit/src/distributedTest/java
> > > and then run it in IntelliJ or however you want.
> > >
> > > It passes, but if you search through the output you'll find the error
> in
> > my
> > > first message with:
> > >
> > > Caused by: java.lang.ClassNotFoundException:
> > > *org.apache.geode.internal.logging.LogService*
> > > at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
> > > at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
> > > at
> sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:349)
> > > at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> > > at
> > >
> > >
> >
> org.eclipse.jetty.webapp.WebAppClassLoader.loadClass(WebAppClassLoader.java:543)
> > > at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> > > ... 88 more
> > >
> > > /*
> > >  * Licensed to the Apache Software Foundation (ASF) under one or more
> > > contributor license
> > >  * agreements. See the NOTICE file distributed with this work for
> > > additional information regarding
> > >  * copyright ownership. The ASF licenses this file to You under the
> > Apache
> > > License, Version 2.0 (the
> > >  * "License"); you may not use this file except in compliance with the
> > > License. You may obtain a
> > >  * copy of the License at
> > >  *
> > >  * http://www.apache.org/licenses/LICENSE-2.0
> > >  *
> > >  * Unless required by applicable law or agreed to in writing, software
> > > distributed under the License
> > >  * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS
> > OF
> > > ANY KIND, either express
> > >  * or implied. See the License for the specific language governing
> > > permissions and limitations under
> > >  * the License.
> > >  */
> > > package org.apache.geode.test.dunit.examples;
> > >
> > > import static
> > > org.apache.geode.distributed.ConfigurationProperties.HTTP_SERVICE_PORT;
> > > import static
> > > org.apache.geode.distributed.ConfigurationProperties.JMX_MANAGER_PORT;
> > > import static org.apache.geode.internal.AvailablePort.SOCKET;
> > > import static
> > > org.apache.geode.internal.AvailablePort.getRandomAvailablePort;
> > > import static org.apache.geode.test.dunit.VM.getVM;
> > > import static org.assertj.core.api.Assertions.assertThat;
> > >
> > > import java.io.Serializable;
> > >
> > > import org.junit.After;
> > > import org.junit.Before;
> > > import org.junit.Rule;
> > > import org.junit.Test;
> > >
> > > import org.apache.geode.distributed.LocatorLauncher;
> > > import org.apache.geode.test.dunit.rules.DistributedRule;
> > > import
> > >
> >
> org.apache.geode.test.junit.rules.serializable.SerializableTemporaryFolder;
> > >
> > > public class LocatorLauncherDistributedTest implements Serializable {
> > >
> > >   private static LocatorLauncher locatorLauncher;
> > >
> > >   @Rule
> > >   public DistributedRule distributedRule = new DistributedRule();
> > >
> > >   @Rule
> > >   public SerializableTemporaryFolder temporaryFolder = new
> > > SerializableTemporaryFolder();
> > >
> > >   @Before
> > >   public void setUp() {
> > > String name = getClass().getSimpleName();
> > > int port = getRandomAvailablePort(SOCKET);
> > >
> > > getVM(0).invoke(() -> {
> > >   locatorLauncher = new LocatorLauncher.Builder()
> > >
> > > .setWorkingDirectory(temporaryFolder.newFolder(name).getAbsolutePath())
> > >   .setMemberName(name)
> > >   .setPort(port)
> > >   .set(HTTP_SERVICE_PORT, "0")
> > >   .set(JMX_MANAGER_PORT, "0")
> > >   .build();
> > >   locatorLauncher.start();
> > > });
> > >   }
> > >
> > >   @After
> > >   public void tearDown() {
> > > getVM(0).invoke(() -> {
> > >   locatorLauncher.stop();
> > > });
> > >   }
> > >
> > >   @Test
> > >   public void isRunning() {
> > > getVM(0).invoke(() -> {
> > >   assertThat(locatorLauncher.isRunning()).isTrue();
> > > });
> > >   }
> > > }
> > >
> > >
> > > On Fri, Nov 1, 2019 at 3:02 PM Kirk 

Re: ***UNCHECKED*** Re: ClassNotFoundException: LogService

2019-11-04 Thread Kirk Lund
If it is picking up an old version of geode then it's doing so because we
use old versions for upgrade/backwards testing. The only old versions of
the jars I have are the ones that our build pulls down for the upgrade
tests. These old versions do tend confuse intellij so it's possible it's a
problem only in intellij. Since I use the debugger a lot, I use IntelliJ a
lot and I believe it's vital we keep IntelliJ working for geode for
everyone.

On Fri, Nov 1, 2019 at 4:16 PM Dan Smith  wrote:

> Hi Kirk,
>
> Did you do a ./gradlew clean? It looks like you might be picking up an old
> version of one of the geode war files that is referring to the old location
> of the LogService class.
>
> -Dan
>
> On Fri, Nov 1, 2019 at 3:28 PM Kirk Lund  wrote:
>
> > To reproduce, drop this new test in geode-dunit/src/distributedTest/java
> > and then run it in IntelliJ or however you want.
> >
> > It passes, but if you search through the output you'll find the error in
> my
> > first message with:
> >
> > Caused by: java.lang.ClassNotFoundException:
> > *org.apache.geode.internal.logging.LogService*
> > at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
> > at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
> > at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:349)
> > at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> > at
> >
> >
> org.eclipse.jetty.webapp.WebAppClassLoader.loadClass(WebAppClassLoader.java:543)
> > at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> > ... 88 more
> >
> > /*
> >  * Licensed to the Apache Software Foundation (ASF) under one or more
> > contributor license
> >  * agreements. See the NOTICE file distributed with this work for
> > additional information regarding
> >  * copyright ownership. The ASF licenses this file to You under the
> Apache
> > License, Version 2.0 (the
> >  * "License"); you may not use this file except in compliance with the
> > License. You may obtain a
> >  * copy of the License at
> >  *
> >  * http://www.apache.org/licenses/LICENSE-2.0
> >  *
> >  * Unless required by applicable law or agreed to in writing, software
> > distributed under the License
> >  * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS
> OF
> > ANY KIND, either express
> >  * or implied. See the License for the specific language governing
> > permissions and limitations under
> >  * the License.
> >  */
> > package org.apache.geode.test.dunit.examples;
> >
> > import static
> > org.apache.geode.distributed.ConfigurationProperties.HTTP_SERVICE_PORT;
> > import static
> > org.apache.geode.distributed.ConfigurationProperties.JMX_MANAGER_PORT;
> > import static org.apache.geode.internal.AvailablePort.SOCKET;
> > import static
> > org.apache.geode.internal.AvailablePort.getRandomAvailablePort;
> > import static org.apache.geode.test.dunit.VM.getVM;
> > import static org.assertj.core.api.Assertions.assertThat;
> >
> > import java.io.Serializable;
> >
> > import org.junit.After;
> > import org.junit.Before;
> > import org.junit.Rule;
> > import org.junit.Test;
> >
> > import org.apache.geode.distributed.LocatorLauncher;
> > import org.apache.geode.test.dunit.rules.DistributedRule;
> > import
> >
> org.apache.geode.test.junit.rules.serializable.SerializableTemporaryFolder;
> >
> > public class LocatorLauncherDistributedTest implements Serializable {
> >
> >   private static LocatorLauncher locatorLauncher;
> >
> >   @Rule
> >   public DistributedRule distributedRule = new DistributedRule();
> >
> >   @Rule
> >   public SerializableTemporaryFolder temporaryFolder = new
> > SerializableTemporaryFolder();
> >
> >   @Before
> >   public void setUp() {
> > String name = getClass().getSimpleName();
> > int port = getRandomAvailablePort(SOCKET);
> >
> > getVM(0).invoke(() -> {
> >   locatorLauncher = new LocatorLauncher.Builder()
> >
> > .setWorkingDirectory(temporaryFolder.newFolder(name).getAbsolutePath())
> >   .setMemberName(name)
> >   .setPort(port)
> >   .set(HTTP_SERVICE_PORT, "0")
> >   .set(JMX_MANAGER_PORT, "0")
> >   .build();
> >   locatorLauncher.start();
> > });
> >   }
> >
> >   @After
> >   public void tearDown() {
> > getVM(0).invoke(() -> {
> >   locatorLauncher.stop();
> > });
> >   }
> >
> >   @Test
> >   public void isRunning() {
> > getVM(0).invoke(() -> {
> >   assertThat(locatorLauncher.isRunning()).isTrue();
> > });
> >   }
> > }
> >
> >
> > On Fri, Nov 1, 2019 at 3:02 PM Kirk Lund  wrote:
> >
> > > I'm seeing ClassNotFoundExceptions for FQN of the old package of the
> > > LogService:
> > >
> > > [locator] Caused by: java.lang.ClassNotFoundException:
> > > *org.apache.geode.internal.logging.LogService*
> > > [locator] at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
> > >
> > > The above is interesting because it's the old package, not the current
> > > one. The current 

Re: ClassNotFoundException: LogService

2019-11-04 Thread Kirk Lund
And yes, I do ./gradlew clean daily despite previously being told not to on
this list.

I will make sure that dunit suspect string grep is working on the
background locator JVM in dunit. If it's not (which I suspect) then it'll
start failing on that stack when I fix suspect-grep.

On Fri, Nov 1, 2019 at 4:22 PM Dan Smith  wrote:

> Your test passes for me, btw.
>
> -Dan
>
> On Fri, Nov 1, 2019 at 4:14 PM Dan Smith  wrote:
>
> > Hi Kirk,
> >
> > Did you do a ./gradlew clean? It looks like you might be picking up an
> old
> > version of one of the geode war files that is referring to the old
> location
> > of the LogService class.
> >
> > -Dan
> >
> > On Fri, Nov 1, 2019 at 3:28 PM Kirk Lund  wrote:
> >
> >> To reproduce, drop this new test in geode-dunit/src/distributedTest/java
> >> and then run it in IntelliJ or however you want.
> >>
> >> It passes, but if you search through the output you'll find the error in
> >> my
> >> first message with:
> >>
> >> Caused by: java.lang.ClassNotFoundException:
> >> *org.apache.geode.internal.logging.LogService*
> >> at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
> >> at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
> >> at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:349)
> >> at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> >> at
> >>
> >>
> org.eclipse.jetty.webapp.WebAppClassLoader.loadClass(WebAppClassLoader.java:543)
> >> at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> >> ... 88 more
> >>
> >> /*
> >>  * Licensed to the Apache Software Foundation (ASF) under one or more
> >> contributor license
> >>  * agreements. See the NOTICE file distributed with this work for
> >> additional information regarding
> >>  * copyright ownership. The ASF licenses this file to You under the
> Apache
> >> License, Version 2.0 (the
> >>  * "License"); you may not use this file except in compliance with the
> >> License. You may obtain a
> >>  * copy of the License at
> >>  *
> >>  * http://www.apache.org/licenses/LICENSE-2.0
> >>  *
> >>  * Unless required by applicable law or agreed to in writing, software
> >> distributed under the License
> >>  * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS
> OF
> >> ANY KIND, either express
> >>  * or implied. See the License for the specific language governing
> >> permissions and limitations under
> >>  * the License.
> >>  */
> >> package org.apache.geode.test.dunit.examples;
> >>
> >> import static
> >> org.apache.geode.distributed.ConfigurationProperties.HTTP_SERVICE_PORT;
> >> import static
> >> org.apache.geode.distributed.ConfigurationProperties.JMX_MANAGER_PORT;
> >> import static org.apache.geode.internal.AvailablePort.SOCKET;
> >> import static
> >> org.apache.geode.internal.AvailablePort.getRandomAvailablePort;
> >> import static org.apache.geode.test.dunit.VM.getVM;
> >> import static org.assertj.core.api.Assertions.assertThat;
> >>
> >> import java.io.Serializable;
> >>
> >> import org.junit.After;
> >> import org.junit.Before;
> >> import org.junit.Rule;
> >> import org.junit.Test;
> >>
> >> import org.apache.geode.distributed.LocatorLauncher;
> >> import org.apache.geode.test.dunit.rules.DistributedRule;
> >> import
> >>
> >>
> org.apache.geode.test.junit.rules.serializable.SerializableTemporaryFolder;
> >>
> >> public class LocatorLauncherDistributedTest implements Serializable {
> >>
> >>   private static LocatorLauncher locatorLauncher;
> >>
> >>   @Rule
> >>   public DistributedRule distributedRule = new DistributedRule();
> >>
> >>   @Rule
> >>   public SerializableTemporaryFolder temporaryFolder = new
> >> SerializableTemporaryFolder();
> >>
> >>   @Before
> >>   public void setUp() {
> >> String name = getClass().getSimpleName();
> >> int port = getRandomAvailablePort(SOCKET);
> >>
> >> getVM(0).invoke(() -> {
> >>   locatorLauncher = new LocatorLauncher.Builder()
> >>
> >> .setWorkingDirectory(temporaryFolder.newFolder(name).getAbsolutePath())
> >>   .setMemberName(name)
> >>   .setPort(port)
> >>   .set(HTTP_SERVICE_PORT, "0")
> >>   .set(JMX_MANAGER_PORT, "0")
> >>   .build();
> >>   locatorLauncher.start();
> >> });
> >>   }
> >>
> >>   @After
> >>   public void tearDown() {
> >> getVM(0).invoke(() -> {
> >>   locatorLauncher.stop();
> >> });
> >>   }
> >>
> >>   @Test
> >>   public void isRunning() {
> >> getVM(0).invoke(() -> {
> >>   assertThat(locatorLauncher.isRunning()).isTrue();
> >> });
> >>   }
> >> }
> >>
> >>
> >> On Fri, Nov 1, 2019 at 3:02 PM Kirk Lund  wrote:
> >>
> >> > I'm seeing ClassNotFoundExceptions for FQN of the old package of the
> >> > LogService:
> >> >
> >> > [locator] Caused by: java.lang.ClassNotFoundException:
> >> > *org.apache.geode.internal.logging.LogService*
> >> > [locator] at
> java.net.URLClassLoader.findClass(URLClassLoader.java:382)
> >> >
> >> > The above is 

Re: ClassNotFoundException: LogService

2019-11-04 Thread Kirk Lund
Yes, it passes for me always, too, as I said.

But the background locator JVM logs that scary stack trace about not
finding LogService in the wrong package. Could you please look more closely?

On Fri, Nov 1, 2019 at 4:22 PM Dan Smith  wrote:

> Your test passes for me, btw.
>
> -Dan
>
> On Fri, Nov 1, 2019 at 4:14 PM Dan Smith  wrote:
>
> > Hi Kirk,
> >
> > Did you do a ./gradlew clean? It looks like you might be picking up an
> old
> > version of one of the geode war files that is referring to the old
> location
> > of the LogService class.
> >
> > -Dan
> >
> > On Fri, Nov 1, 2019 at 3:28 PM Kirk Lund  wrote:
> >
> >> To reproduce, drop this new test in geode-dunit/src/distributedTest/java
> >> and then run it in IntelliJ or however you want.
> >>
> >> It passes, but if you search through the output you'll find the error in
> >> my
> >> first message with:
> >>
> >> Caused by: java.lang.ClassNotFoundException:
> >> *org.apache.geode.internal.logging.LogService*
> >> at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
> >> at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
> >> at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:349)
> >> at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> >> at
> >>
> >>
> org.eclipse.jetty.webapp.WebAppClassLoader.loadClass(WebAppClassLoader.java:543)
> >> at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> >> ... 88 more
> >>
> >> /*
> >>  * Licensed to the Apache Software Foundation (ASF) under one or more
> >> contributor license
> >>  * agreements. See the NOTICE file distributed with this work for
> >> additional information regarding
> >>  * copyright ownership. The ASF licenses this file to You under the
> Apache
> >> License, Version 2.0 (the
> >>  * "License"); you may not use this file except in compliance with the
> >> License. You may obtain a
> >>  * copy of the License at
> >>  *
> >>  * http://www.apache.org/licenses/LICENSE-2.0
> >>  *
> >>  * Unless required by applicable law or agreed to in writing, software
> >> distributed under the License
> >>  * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS
> OF
> >> ANY KIND, either express
> >>  * or implied. See the License for the specific language governing
> >> permissions and limitations under
> >>  * the License.
> >>  */
> >> package org.apache.geode.test.dunit.examples;
> >>
> >> import static
> >> org.apache.geode.distributed.ConfigurationProperties.HTTP_SERVICE_PORT;
> >> import static
> >> org.apache.geode.distributed.ConfigurationProperties.JMX_MANAGER_PORT;
> >> import static org.apache.geode.internal.AvailablePort.SOCKET;
> >> import static
> >> org.apache.geode.internal.AvailablePort.getRandomAvailablePort;
> >> import static org.apache.geode.test.dunit.VM.getVM;
> >> import static org.assertj.core.api.Assertions.assertThat;
> >>
> >> import java.io.Serializable;
> >>
> >> import org.junit.After;
> >> import org.junit.Before;
> >> import org.junit.Rule;
> >> import org.junit.Test;
> >>
> >> import org.apache.geode.distributed.LocatorLauncher;
> >> import org.apache.geode.test.dunit.rules.DistributedRule;
> >> import
> >>
> >>
> org.apache.geode.test.junit.rules.serializable.SerializableTemporaryFolder;
> >>
> >> public class LocatorLauncherDistributedTest implements Serializable {
> >>
> >>   private static LocatorLauncher locatorLauncher;
> >>
> >>   @Rule
> >>   public DistributedRule distributedRule = new DistributedRule();
> >>
> >>   @Rule
> >>   public SerializableTemporaryFolder temporaryFolder = new
> >> SerializableTemporaryFolder();
> >>
> >>   @Before
> >>   public void setUp() {
> >> String name = getClass().getSimpleName();
> >> int port = getRandomAvailablePort(SOCKET);
> >>
> >> getVM(0).invoke(() -> {
> >>   locatorLauncher = new LocatorLauncher.Builder()
> >>
> >> .setWorkingDirectory(temporaryFolder.newFolder(name).getAbsolutePath())
> >>   .setMemberName(name)
> >>   .setPort(port)
> >>   .set(HTTP_SERVICE_PORT, "0")
> >>   .set(JMX_MANAGER_PORT, "0")
> >>   .build();
> >>   locatorLauncher.start();
> >> });
> >>   }
> >>
> >>   @After
> >>   public void tearDown() {
> >> getVM(0).invoke(() -> {
> >>   locatorLauncher.stop();
> >> });
> >>   }
> >>
> >>   @Test
> >>   public void isRunning() {
> >> getVM(0).invoke(() -> {
> >>   assertThat(locatorLauncher.isRunning()).isTrue();
> >> });
> >>   }
> >> }
> >>
> >>
> >> On Fri, Nov 1, 2019 at 3:02 PM Kirk Lund  wrote:
> >>
> >> > I'm seeing ClassNotFoundExceptions for FQN of the old package of the
> >> > LogService:
> >> >
> >> > [locator] Caused by: java.lang.ClassNotFoundException:
> >> > *org.apache.geode.internal.logging.LogService*
> >> > [locator] at
> java.net.URLClassLoader.findClass(URLClassLoader.java:382)
> >> >
> >> > The above is interesting because it's the old package, not the current
> >> > one. The current 

Lucene upgrade

2019-11-04 Thread Mario Kevo
Hi geode dev,

I'm working on upgrade lucene to a newer version. (
https://issues.apache.org/jira/browse/GEODE-7309)

I followed instruction from 
https://cwiki.apache.org/confluence/display/GEODE/Upgrading+to+Lucene+7.1.0
Also add some other changes that is needed for lucene 8.2.0.

I found some problems with tests:
 * geode-
   lucene/src/test/java/org/apache/geode/cache/lucene/internal/distribu
   ted/DistributedScoringJUnitTest.java:  


 * 
geode-lucene/src/upgradeTest/java/org/apache/geode/cache/lucene/RollingUpgradeQueryReturnsCorrectResultsAfterClientAndServersAreRolledOver.java:
 * 
geode-lucene/src/upgradeTest/java/org/apache/geode/cache/lucene/RollingUpgradeQueryReturnsCorrectResultAfterTwoLocatorsWithTwoServersAreRolled.java:
 * 
./geode-lucene/src/upgradeTest/java/org/apache/geode/cache/lucene/RollingUpgradeQueryReturnsCorrectResultsAfterServersRollOverOnPartitionRegion.java:
 * 
./geode-lucene/src/upgradeTest/java/org/apache/geode/cache/lucene/RollingUpgradeQueryReturnsCorrectResultsAfterServersRollOverOnPersistentPartitionRegion.java:

  -> failed due to 
Caused by: org.apache.lucene.index.IndexFormatTooOldException: Format
version is not supported (resource
BufferedChecksumIndexInput(segments_1)): 6 (needs to be between 7 and
9). This version of Lucene only supports indexes created with release
6.0 and later.
at
org.apache.lucene.codecs.CodecUtil.checkHeaderNoMagic(CodecUtil.java:21
3)
at
org.apache.lucene.index.SegmentInfos.readCommit(SegmentInfos.java:305)
at
org.apache.lucene.index.SegmentInfos.readCommit(SegmentInfos.java:289)
at
org.apache.lucene.index.IndexWriter.(IndexWriter.java:846)
at
org.apache.geode.cache.lucene.internal.IndexRepositoryFactory.finishCom
putingRepository(IndexRepositoryFactory.java:123)
at
org.apache.geode.cache.lucene.internal.IndexRepositoryFactory.computeIn
dexRepository(IndexRepositoryFactory.java:66)
at
org.apache.geode.cache.lucene.internal.PartitionedRepositoryManager.com
puteRepository(PartitionedRepositoryManager.java:151)
at
org.apache.geode.cache.lucene.internal.PartitionedRepositoryManager.lam
bda$computeRepository$1(PartitionedRepositoryManager.java:170)
... 16 more


 * 
geode-lucene/src/upgradeTest/java/org/apache/geode/cache/lucene/RollingUpgradeQueryReturnsCorrectResultsAfterClientAndServersAreRolledOverAllBucketsCreated.java:
 
  -> failed with the same exception as previous tests


I found this on web 
https://stackoverflow.com/questions/47454434/solr-indexing-issue-after-upgrading-from-4-7-to-7-1
, but not have an idea how to proceed with that.

Does anyone has any idea how to fix it?

BR,
Mario