Re: Our JIRA is fully functional now

2015-05-06 Thread Jacob Barrett
jbarrett — Jacob Barrett Enterprise Architect Pivotal jbarr...@pivotal.io 503-533-3763 On Tue, May 5, 2015 at 4:36 PM, Roman Shaposhnik wrote: > https://issues.apache.org/jira/browse/GEODE-1 > Please make sure to register accounts there > and reply back to this thread with

Re: [jira] [Created] (GEODE-22) Create a parent POM

2015-05-11 Thread Jacob Barrett
parent as some may not be applicable, like Hydra or something. — Jacob Barrett Enterprise Architect Pivotal jbarr...@pivotal.io 503-533-3763 On Mon, May 11, 2015 at 1:36 PM, Anthony Baker (JIRA) wrote: > Anthony Baker created GEODE

Re: [rtds-dev] What to call Apache Geode's command-line shell tool...

2015-05-20 Thread Jacob Barrett
gsh - G SHell - Geode SHell (my vote) gash - Geode Again SHell (nod to bash) gctl - Geode ConTroL — Jacob Barrett Enterprise Architect Pivotal jbarr...@pivotal.io 503-533-3763 On Wed, May 20, 2015 at 5:16 PM, Wes Williams wrote: > The problem with leaving it as &q

Re: [jira] [Commented] (GEODE-52) Remove @author tags from Java source

2015-06-05 Thread Jacob Barrett
+1 — Jacob Barrett Manager GemFire Advanced Customer Engineering (ACE) Pivotal jbarr...@pivotal.io 503-533-3763 For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Fri, Jun 5, 2015 at 3:57 PM, John Blum (JIRA) wrote: > [ >

Re: another cleanup JIRA

2015-07-01 Thread Jacob Barrett
I think you meant https://issues.apache.org/jira/browse/GEODE-73 — Jacob Barrett Manager GemFire Advanced Customer Engineering (ACE) Pivotal jbarr...@pivotal.io 503-533-3763 For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Wed, Jul 1, 2015 at

Re: Review Request 36397: HdfsStore API refactoring

2015-07-13 Thread Jacob Barrett
+1 We need to fail reviews that don’t have unit tests. There is a handy field on the review board for “Testing Done” which should probably used to call out the unit tests for the reviewer to run. -Jake — Jacob Barrett Manager GemFire Advanced Customer Engineering (ACE) Pivotal

Re: Review Request 36906: GEODE-150 Implemented merge function for Aggregator interface. Made the aggregators DataSerializable

2015-07-30 Thread Jacob Barrett
ere is what it would look like. ... As XML: com.example.foo.Foo some string com.example.foo.SomeOtherClass - Jacob Barrett On July 29

Re: Review Request 36906: GEODE-150 Implemented merge function for Aggregator interface. Made the aggregators DataSerializable

2015-07-30 Thread Jacob Barrett
now. As part of initialization of UDAs , we > do not need instance of this class. The instances are created during query > execution time. > In which case , how will the xsd look? > Regards > Asif > On Thu, Jul 30, 2015 at 7:43 PM, Jacob Barrett wrote: >> This is an automatically gene

Re: Nabble for Geode mail archives?

2015-08-06 Thread Jacob Barrett
website direct people to something more useful? — Jacob Barrett Manager GemFire Advanced Customer Engineering (ACE) Pivotal jbarr...@pivotal.io 503-533-3763 For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Thu, Aug 6, 2015 at 4:04 PM, William Markito

Re: Review Request 37209: GEODE-17 : Integrated Security Code Merge

2015-08-07 Thread Jacob Barrett
rarely want to reinvent something, especially in security. -Jake — Jacob Barrett Manager GemFire Advanced Customer Engineering (ACE) Pivotal jbarr...@pivotal.io 503-533-3763 For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Fri, Aug 7, 2015

Re: YCSB driver for Geode

2015-08-13 Thread Jacob Barrett
Do you have a corresponding error message or indicator of it not being compatible? — Jacob Barrett Manager GemFire Advanced Customer Engineering (ACE) Pivotal jbarr...@pivotal.io 503-533-3763 For immediate support please contact Pivotal Support at http://support.pivotal.io/ On

Re: YCSB driver for Geode

2015-08-13 Thread Jacob Barrett
a-instance";     xsi:schemaLocation="http://schema.pivotal.io/gemfire/cache http://schema.pivotal.io/gemfire/cache/cache-8.1.xsd";     version="8.1”> — Jacob Barrett Manager GemFire Advanced Customer Engineering (ACE) Pivotal jbarr...@pivotal.io 503-533-3763 For immedia

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

2015-08-22 Thread Jacob Barrett
will be implied. ```java method.invoke(null); ``` is the same as ```java method.invoke(null, (Object[]) null); ``` but in my opinion a little more readable. - Jacob Barrett On Aug. 22, 2015, 9:03 a.m., Anthony Baker wrote: > > ---

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

2015-08-24 Thread Jacob Barrett
> On Aug. 22, 2015, 9:39 a.m., Jacob Barrett wrote: > > You can leave off the vararg argument completely as well. The null will be > > implied. > > ```java > > method.invoke(null); > > ``` > > is the same as > > ```java > > method.invoke(null

Re: Using JSON to create Regions Dynamically

2015-08-31 Thread Jacob Barrett
You have to quote all your property names and remove the // comments. Jacob Barrett  Manager  GemFire Advanced Customer Engineering (ACE)  Pivotal jbarr...@pivotal.io  503-533-3763 For immediate support please contact Pivotal Support at  http://support.pivotal.io/ On Mon, Aug 31, 2015 at 11

Re: GEODE-218: Dunit Fork Every 1

2015-09-04 Thread Jacob Barrett
Can any of them be launched in parallel? This could cut down time significantly. Jacob Barrett  Manager  GemFire Advanced Customer Engineering (ACE)  Pivotal jbarr...@pivotal.io  503-533-3763 For immediate support please contact Pivotal Support at  http://support.pivotal.io/ On Fri, Sep 4

Re: GEODE-327: Rename gemfire-junit to gemfire-common

2015-09-11 Thread Jacob Barrett
-1 Reserve common for things common to geode development not related to unit testing. Like utilities classes. Jacob Barrett  Manager  GemFire Advanced Customer Engineering (ACE)  Pivotal jbarr...@pivotal.io  503-533-3763 For immediate support please contact Pivotal Support at  http

Re: GEODE-327: Rename gemfire-junit to gemfire-common

2015-09-12 Thread Jacob Barrett
re at runtime, like string utils, logging, and other cross cutting runtime concerns. If you want a library for common test classes then think gemfire-test-common.jar or something. Jacob Barrett  Manager  GemFire Advanced Customer Engineering (ACE)  Pivotal jbarr...@pivotal.io  503-533-3763

Re: GEODE-327: Rename gemfire-junit to gemfire-common

2015-09-12 Thread Jacob Barrett
+1 Jacob Barrett  Manager  GemFire Advanced Customer Engineering (ACE)  Pivotal jbarr...@pivotal.io  503-533-3763 For immediate support please contact Pivotal Support at  http://support.pivotal.io/ On Sat, Sep 12, 2015 at 11:16 AM, Kirk Lund wrote: > I'm pretty sure we coul

Re: Review Request 39110: Adding xml parsing and generation logic

2015-10-08 Thread Jacob Barrett
nt on breaking fields into collection of field elements. - Jacob Barrett On Oct. 7, 2015, 4:34 p.m., Dan Smith wrote: > > --- > This is an automatically generated e-mail. To r

Re: Review Request 39107: Adding an EntityResolver for the apache namespace

2015-10-08 Thread Jacob Barrett
/internal/cache/xmlcache/DefaultEntityResolver2.java (line 53) <https://reviews.apache.org/r/39107/#comment159440> DOH! Thanks! Looks great! - Jacob Barrett On Oct. 7, 2015, 4:17 p.m., Dan Smith wrote: > > --- > This is a

Re: xml schema locations

2015-10-08 Thread Jacob Barrett
Does the ASF have a central schema respository we can use? Jacob Barrett  Manager  GemFire Advanced Customer Engineering (ACE)  Pivotal jbarr...@pivotal.io  503-533-3763 For immediate support please contact Pivotal Support at  http://support.pivotal.io/ On Thu, Oct 8, 2015 at 11:41 AM

Re: Review Request 39195: GEODE-331 Bump Jetty version to 9.2.13.v20150730

2015-10-09 Thread Jacob Barrett
I think we had a discussion about dropping 1.7 support in geode. On Fri, Oct 9, 2015 at 4:32 PM, Jens Deppe wrote: >> On Oct. 9, 2015, 11:11 p.m., William Markito wrote: >> > Hi Jens! Why not go to 9.3 ? > 9.3 is JDK8 only > - Jens > --- >

Re: Review Request 39279: GEODE-397: Fix clients to use server ssl config

2015-10-13 Thread Jacob Barrett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39279/#review102493 --- Ship it! Ship It! - Jacob Barrett On Oct. 13, 2015, 10:07 a.m

Re: Review Request 39278: GEODE-396: Fix server to use server/gateway ssl config

2015-10-13 Thread Jacob Barrett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39278/#review102494 --- Ship it! Ship It! - Jacob Barrett On Oct. 13, 2015, 9:44 a.m

Re: Review Request 39328: Adding an internal listener that can modify region attributes

2015-10-15 Thread Jacob Barrett
/GemFireCacheImpl.java (line 167) <https://reviews.apache.org/r/39328/#comment160573> Where is this class in use? - Jacob Barrett On Oct. 15, 2015, 10:18 a.m., Dan Smith wrote: > > --- > This is an automatically gener

Re: Review Request 39328: Adding an internal listener that can modify region attributes

2015-10-15 Thread Jacob Barrett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39328/#review102798 --- Great addition to the extensions!! - Jacob Barrett On Oct. 15

Re: Bump language level to 1.8

2015-10-23 Thread Jacob Barrett
+1 All for dropping unsupported Java runtimes! On Fri, Oct 23, 2015 at 9:27 AM, William Markito wrote: > Folks, > We had some discussions around this in the past but I guess we have not yet > made the decision to move forward and change the language level. > Just opened GEODE-479 and would l

Re: incubator-geode git commit: GEM-111: Cannot use 8.0.0.9+ clients with 8.1+ servers Added support for version 8009

2015-11-10 Thread Jacob Barrett
This should have a GEODE ticket created for this work. — Jacob Barrett  Manager  GemFire Advanced Customer Engineering (ACE)  Pivotal jbarr...@pivotal.io  503-533-3763 For immediate support please contact Pivotal Support at  http://support.pivotal.io/ On Tue, Nov 10, 2015 at 11:41 AM, Mark

Re: License headers on new source files

2015-11-24 Thread Jacob Barrett
There are plugins for gradle that check for copyright and license headers. They can be configured to fail the build if they are missing from any files. It would be a good way to make sure all files have the headers. -Jake — Jacob Barrett  Manager  GemFire Advanced Customer

Re: removing the license from jgroups configuration files

2015-12-04 Thread Jacob Barrett
Do we really need to be adding licensing and copyright notices to config files? — Jacob Barrett  Manager  GemFire Advanced Customer Engineering (ACE)  Pivotal jbarr...@pivotal.io  503-533-3763 For immediate support please contact Pivotal Support at  http://support.pivotal.io/ On Fri, Dec 4

Re: removing the license from jgroups configuration files

2015-12-04 Thread Jacob Barrett
x27;s creativity, add the license header to the file. I would say configuration files lack any creativity. — Jacob Barrett  Manager  GemFire Advanced Customer Engineering (ACE)  Pivotal jbarr...@pivotal.io  503-533-3763 For immediate support please contact Pivotal Support at 

Re: Licenses on XML files

2015-12-07 Thread Jacob Barrett
The code should be looking for the first element, not first node. Alternately the DOM parser can be configured to ignore comments.  — Jacob Barrett  Manager  GemFire Advanced Customer Engineering (ACE)  Pivotal jbarr...@pivotal.io  503-533-3763 For immediate support please contact Pivotal

Re: Review Request 41803: GEODE-718: Sanitize passwords in gfsh history file

2016-01-04 Thread Jacob Barrett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41803/#review112592 --- Ship it! Ship It! - Jacob Barrett On Dec. 30, 2015, 10:57 a.m

Re: Deprecating subregions

2016-01-07 Thread Jacob Barrett
The concept of "deprecating" need to be rethought. Deprecating something for an entire major release isn't helpful to anyone. The user still expects it to work. Developers give it no love or attention. Things break. Then the next release when it is pulled the user has to update their code anyway. I

Re: Can we tweak svngit2jira to ignore feature branches ?

2016-01-08 Thread Jacob Barrett
++1 On Fri, Jan 8, 2016 at 5:06 PM Dan Smith wrote: > ++1 > > We're currently getting lots of silly comments on tickets, Especially when > downmerging from develop to a feature branch. See for example GEODE-364. It > two messages with the same hash. The second one is reporting that the > commit

Can we change JIRA notifications?

2016-01-08 Thread Jacob Barrett
In line with William's request to limit the JIRA emails on feature branches, can we change the notification model for all JIRAs to something less frequent. Only have the initial create notification go to the entire group. After that only those "watching" the JIRA will get notifications. If you wan

Re: Previous versions dtds

2016-09-29 Thread Jacob Barrett
I think you should drop all the GemFire DTDs from Geode. Geode doesn't need to be tainted with reading old GemFire configuration files. It is very simple for someone to update their XML to the Geode XSD. On Thu, Sep 29, 2016 at 2:15 PM Darrel Schneider wrote: > statisticsType.dtd is used > by

Re: Limiting scope for Geode 1.0

2016-09-29 Thread Jacob Barrett
+1 for creating branch now to prevent feature creep. On Thu, Sep 29, 2016 at 2:10 PM Kirk Lund wrote: > I think we should propose creating that release branch sooner (now?) so we > can minimize unplanned changes slipping into 1.0 and destabilizing it. > > -Kirk > > On Thursday, September 29, 201

Re: Previous versions dtds

2016-09-29 Thread Jacob Barrett
The files MUST remain in the source otherwise the validating parser will have to download them from the website. This is VERY slow and only works if you have Internet access. The schema resolver first looks in the class path then to the web. -Jake On Thu, Sep 29, 2016 at 2:24 PM Anthony Baker w

Re: backward compatibility testing against GemFire 8.x WAN/clients

2016-10-06 Thread Jacob Barrett
-1 Completely agree with Mark on this. On Thu, Oct 6, 2016 at 9:31 AM Mark Bretl wrote: > -1 for this > > I do understand the intent and reason behind this, however, in addition to > the reasons Anthony provided, I do not believe this community should carry > the burden of testing compatibility

Re: Review Request 52641: GEODE-1914 removed old versions of dtds

2016-10-07 Thread Jacob Barrett
If were are in there changing them all why wouldn't we just change them all to the latest xsd? On Fri, Oct 7, 2016 at 10:56 AM Hitesh Khamesra wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.ap

Re: Coding practices/standards

2016-10-14 Thread Jacob Barrett
+1 On Thu, Oct 13, 2016 at 10:04 AM Kevin Duling wrote: > Given that, +1 from me! > > On Thu, Oct 13, 2016 at 9:51 AM, Jared Stewart > wrote: > > > The task is fully suppressible with -x spotlessCheck. Also, if you have > > any formatter errors you can automatically fix them with 'gradle > > s

Re: GEODE-88: code fixes for c++ client

2016-10-20 Thread Jacob Barrett
See comments on pull request. On Wed, Oct 19, 2016 at 11:44 PM Dor Ben Dov wrote: > Hi Everyone, > > What is the status of this pull request? > > Why isn't it merged? > > > Thanks, > Dor > > This message and the information contained herein is proprietary and > confidential and subject to the A

Re: native-client-software-grant - ClientMetadata::getServerLocation

2016-10-20 Thread Jacob Barrett
Please provide a link to the pull request. On Wed, Oct 19, 2016 at 11:34 PM Dor Ben Dov wrote: > Hi Everyone, > > What is the status of this pull request? > > > > Thanks, > Dor > > This message and the information contained herein is proprietary and > confidential and subject to the Amdocs polic

C++ Coding Standards

2016-10-21 Thread Jacob Barrett
Considering the team just formally adopted some standards and enforcement for the Java sources I would like to open the discussion for formally adopting similar standards for the C++ sources. I propose that we use the Google C++ style as defined by https://google.github.io/styleguide/cppguide.html

Re: Where should geode example code live

2016-11-10 Thread Jacob Barrett
+1 On Thu, Nov 10, 2016 at 11:22 AM Swapnil Bawaskar wrote: > +1 > > On Thu, Nov 10, 2016 at 11:12 AM, Kirk Lund wrote: > > > Yep! Replace "gemfire" with "geode" > > > > On Thu, Nov 10, 2016 at 11:07 AM, Kevin Duling > wrote: > > > > > +1 > > > > > > Shouldn't that be org.apache.geode.security

Re: Should jca artifact be a rar?

2016-02-05 Thread Jacob Barrett
If it is a fully packaged rar it should be a rar (manifest, dependencies, etc.). If however it is just the classes it should be a jar. On Fri, Feb 5, 2016 at 3:24 PM Jens Deppe wrote: > As the subject indicates, is this correct or should the jca artifact really > be a jar? > > > https://github.c

Re: AvailablePort.getRandomAvailablePort and CI failures

2016-02-09 Thread Jacob Barrett
+1 I am dealing with the same issue on some tests on the C++ side that do the same thing. I had planned to fix it the same way that you proposed. Given that there are lots of ports to configure in Geode maybe as those ports are activated they should be written to a properties file in the working

Re: AvailablePort.getRandomAvailablePort and CI failures

2016-02-09 Thread Jacob Barrett
2016 at 2:39 PM, Kirk Lund wrote: > > > > > For the C++ client tests, are the ports being used in the Java Server > and > > > Locator? > > > > > > -Kirk > > > > > > > > > On Tue, Feb 9, 2016 at 2:01 PM, Jacob Barrett > >

Re: Reminder: include GEODE-xxx jira ticket # in commit message

2016-02-29 Thread Jacob Barrett
+1 All changes in the repo should have a ticket. On Mon, Feb 29, 2016 at 11:21 AM Udo Kohlmeyer wrote: > My opinion is that no work should be done without a JIRA. That way there > is a "documentation" on what the task is and you can measure the outcome > based on the JIRA. > > One might think t

Re: MultiAxisChartFX and jvsd

2016-05-09 Thread Jacob Barrett
A complete rewrite of MultiAxisChartFX is not planned. The MultiAxisChartFX is derived from the OpenJDK XYChart and carries the same GPL with class path exception that the JDK does. The plan is to merge jVSD into the develop branch with the dependency as is. The binary distribution will not contain

Re: MultiAxisChartFX and jvsd

2016-05-10 Thread Jacob Barrett
ferent solution maybe ? > > Thanks > Dor > -Original Message- > From: Daniel Farcovich > Sent: יום ג 10 מאי 2016 08:37 > To: Jacob Barrett > Cc: dev@geode.incubator.apache.org > Subject: RE: MultiAxisChartFX and jvsd > > Hi Jake, > Do you know when i

Re: MultiAxisChartFX and jvsd

2016-05-10 Thread Jacob Barrett
lop / trunk > branch and continue being apache license 2 - without any dependency issues. > Again, it's not 5min solution of course. > Dor > > -Original Message- > From: Jacob Barrett [mailto:jbarr...@pivotal.io] > Sent: יום ג 10 מאי 2016 19:47 > To: dev@

Re: MultiAxisChartFX and jvsd

2016-05-10 Thread Jacob Barrett
Tue, May 10, 2016 at 9:53 AM Jacob Barrett wrote: > Are you suggesting that it be a desktop app rendering HTML 5 / JS or a web > application? > > On Tue, May 10, 2016 at 9:49 AM Dor Ben Dov > wrote: > >> Maybe replacing it with html 5, I am not saying it's 5 min wo

Re: MultiAxisChartFX and jvsd

2016-05-10 Thread Jacob Barrett
license attached to it. This is the reason this code was in a separate > repo. A > user would need to build that separately and then reference it in this > project." > > Hope this helps. > Best regards." > > Unless I am missing something, there is a contradiction be

Re: how to build native client on linux

2016-05-23 Thread Jacob Barrett
The initial software grant of Native Client is not going to compile out of the box. The build process has dependencies on items that are not publicly accessible at this time. As we replace the old build process with a new process that is not dependent on internal resources we will update the source

Re: c++ native client memory management

2016-06-01 Thread Jacob Barrett
We do not use an external memory manager. On Tue, May 31, 2016 at 8:14 AM Gal Palmery wrote: > Hi, > > Do you use a specific external memory manager (like mtmalloc, hoard, > etc..) for native-client code? > > Thanks, > Gal > > This message and the information contained herein is proprietary and

Re: c++ native client

2016-06-03 Thread Jacob Barrett
The native client in geode will likely be v1 to match the Java bits, or whatever geode version is when nc is ready to release with geode. On Fri, Jun 3, 2016 at 7:18 AM Dor Ben Dov wrote: > Hi, > > Anyone can tell me what is the version of the native client (c++) ? > > Dor > > > This message and

Re: c++ native client

2016-06-05 Thread Jacob Barrett
urrent native client (before > your switched it to the open source) was 7.0.1. > Is this the same 7.0.1 ? > > Dor > > -Original Message----- > From: Jacob Barrett [mailto:jbarr...@pivotal.io] > Sent: יום ו 03 יוני 2016 17:24 > To: dev@geode.incubator.apache.org > Su

Re: update website for WAN, CQ and native client

2016-06-16 Thread Jacob Barrett
-1 I would not mention native clients yet since they aren't anywhere near a state to be built and used. On Thu, Jun 16, 2016 at 5:32 AM yang theseus wrote: > +1 great! > > 2016-06-16 15:14 GMT+08:00 Gregory Chase : > > > +1 > > > > On Wed, Jun 15, 2016 at 11:27 PM, Swapnil Bawaskar > > > wrote:

Re: Snappy compressor (native->java)

2016-06-20 Thread Jacob Barrett
+1 for pure java default and making the native a drop in option. I would suggest looking into embedding the native bits into a JAR file. There are some tricks you can do to write the native bits out to disk from the JAR file to load them at runtime. This would make it easier for someone to deploy

Re: Unit testing classes that depend on static collaborators

2016-07-25 Thread Jacob Barrett
Security is one of those exceptions to the rules about ThreadLocal. Almost every implementation uses ThreadLocal to stash the current executing context. Generally then there is a static class that gets the current context. To test you should be able to just push your own mocked context into the con