---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45919/
---
Review request for geode, Joey McAllister, Swapnil Bawaskar, and Dan Smith.
Rep
GitHub user joeymcallister opened a pull request:
https://github.com/apache/incubator-geode/pull/126
Feature/geode 904
For GEODE-904, this:
- Adds the additional bundle listings to the LICENSES.
- Reformats the LICENSE files around the embedded licenses.
You can merge this p
Done [1].
Nitin
[1] https://issues.apache.org/jira/browse/INFRA-11619
On Wed, Apr 6, 2016 at 7:05 PM, Roman Shaposhnik
wrote:
> On Wed, Apr 6, 2016 at 5:36 PM, Nitin Lamba wrote:
> > +1
> >
> > Another option (mentioned in today's Geode Clubhouse), is to have Geode
> at
> > http://analysis.apa
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45844/#review127703
---
Ship it!
Ship It!
- Jens Deppe
On April 6, 2016, 11:39 p.m.,
If you've had problems with IntelliJ + Gradle correctly resolving test code
as a dependency in other sub-modules I think I've found (and committed a
fix).
Specifically, the geode-core-test.jar is necessary to build test code in
almost all other submodules. When importing the gradle project, Intell
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45706/#review127692
---
geode-core/src/main/java/com/gemstone/gemfire/internal/offheap/Re
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45706/#review127690
---
geode-core/src/main/java/com/gemstone/gemfire/internal/offheap/Re
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45706/
---
(Updated April 7, 2016, 11:28 p.m.)
Review request for geode, Darrel Schneider
3 of these dependencies should now be gone with the changes for GEODE-1025.
spring-context-support
spring-data-gemfire
spring-data-commons
BTW, if you want to see the list of jars we ship, there's a handy gradle
task I added. It will show you what is in each war file as well as the lib
directory:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45844/#review127686
---
Ship it!
Ship It!
- Jianxia Chen
On April 6, 2016, 11:39 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45880/#review127669
---
Fix it, then Ship it!
geode-lucene/src/test/java/com/gemstone/
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45897/
---
Review request for geode, anilkumar gingade, Jianxia Chen, Qihong Chen, and Dan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45881/#review127668
---
Ship it!
Ship It!
- Dan Smith
On April 7, 2016, 5:34 p.m., a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45884/#review127666
---
Ship it!
Looks good! It would probably be good to verify that t
Build Update for apache/incubator-geode
-
Build: #317
Status: Passed
Duration: 16 minutes and 44 seconds
Commit: f2eeebb (keys)
Author: Jens Deppe
Message: Adding GPG key for Jens Deppe
View the changeset:
https://github.com/apache/incubator-geode/commit/f2ee
---
Spring Data GemFire > Nightly-ApacheGeode > #272 was successful.
---
Scheduled
1383 tests in total.
https://build.spring.io/browse/SGF-NAG-272/
--
This
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45879/#review127642
---
Ship it!
Ship It!
- Sai Boorlagadda
On April 7, 2016, 6:43 p
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45844/#review127639
---
Ship it!
Ship It!
- Udo Kohlmeyer
On April 6, 2016, 11:39 p.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45879/
---
(Updated April 7, 2016, 6:43 p.m.)
Review request for geode, Anthony Baker, Wil
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45884/
---
Review request for geode and Dan Smith.
Repository: geode
Description
---
Karen,
Looks good to me.
Just a suggestion to continue with the flow of instructions,
Do we need to point user to the gfsh available in the bin folder of the
extracted binary distribution?
As we instructed about extracting the binary and add the bin directory to
the path.
Sai
On Tue, Apr 5, 201
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45879/#review127632
---
Ship it!
Ship It!
- Sai Boorlagadda
On April 7, 2016, 5:01 p
Added JIRA:
https://issues.apache.org/jira/browse/GEODE-1193
Anthony
> On Apr 7, 2016, at 9:44 AM, Jens Deppe wrote:
>
> The reason that the servlet API jar is included is because it gets used on
> the server classpath (where the sessions are stored). I'm going through the
> Modules docs now, s
GitHub user gemzdude opened a pull request:
https://github.com/apache/incubator-geode/pull/125
GEODE-235: Add region name to EntryEventImpl.toString
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gemzdude/incubator-geode feature
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45881/
---
Review request for geode, anilkumar gingade, Barry Oglesby, Jason Huynh,
nabarun
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45880/
---
Review request for geode, anilkumar gingade, Barry Oglesby, nabarun nag, Dan
Smi
Github user sboorlagadda commented on the pull request:
https://github.com/apache/incubator-geode/pull/122#issuecomment-207004440
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this featu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45819/#review127618
---
Ship it!
Ship It!
- Dan Smith
On April 7, 2016, 2:31 p.m., J
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45879/
---
Review request for geode, Anthony Baker, William Markito, and Sai Boorlagadda.
The reason that the servlet API jar is included is because it gets used on
the server classpath (where the sessions are stored). I'm going through the
Modules docs now, so we should be able to direct users to use the servlet
jar provided by the container. We should be OK not to ship it.
--Jens
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45776/#review127606
---
geode-core/src/main/java/com/gemstone/gemfire/internal/size/Refle
Fixed Dave. Sorry about that.
On Thu, Apr 7, 2016 at 8:57 AM Dave Barnes wrote:
> I can create JIRA tickets and comment on them, but am unable to assign a
> ticket (even to myself). Would someone be so kind as to enlarge my scope?
> My Apache JIRA ID is 'dbarnes97'.
> Thanks,
> -Dave
>
--
~/Wi
I can create JIRA tickets and comment on them, but am unable to assign a
ticket (even to myself). Would someone be so kind as to enlarge my scope?
My Apache JIRA ID is 'dbarnes97'.
Thanks,
-Dave
I would not include the (javax.)servlet-api JAR with Geode. It will be a
significant pain point for web applications (and frameworks, e.g. *Spring
Session*) that expect a specific version of the Servlet API and then
suddenly encounter conflicting versions.
Also, FYI, the latest release of the core
You can’t tell from the diff below but it turns out we’re including two
versions of servlet-api:
servlet-api-2.5.jar
javax.servlet-api-3.1.0.jar
Version 3.1.0 is from the geode/lib while the older jar is from the app server
module. Do we need to ship these at all since they are only needed for
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45819/
---
(Updated April 7, 2016, 2:31 p.m.)
Review request for geode, Anthony Baker, Jin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45819/
---
(Updated April 7, 2016, 2:31 p.m.)
Review request for geode, Anthony Baker, Jin
37 matches
Mail list logo