Re: [Geotools-devel] jdbc-ng event patch now ready

2009-05-10 Thread Andrea Aime
Justin Deoliveira ha scritto: > I am a little confused... the patch does not seem to touch anything > related to id handling. Am I missing something. > > As for null being the placeholder feature id, are you talking about in > REsultSetFeature? What do you want to change it to? > > Id generatio

[Geotools-devel] Hudson build is back to normal: geotools-trunk #1566

2009-05-10 Thread Hudson
See http://hudson.opengeo.org/hudson/job/geotools-trunk/1566/changes -- The NEW KODAK i700 Series Scanners deliver under ANY circumstances! Your production scanning environment may not be a perfect world - but thanks to

Re: [Geotools-devel] JMapPane stuff

2009-05-10 Thread Michael Bedward
Hi Jody, > I would like to hear back on your experience with reordering > MapContext layers. > I've got something working now - still very draft code... In branches/2.6.x_JMapPane_move/demo/mapwidget - I've added a class MapLayerTable which works with a JMapPane object and displays a list of lay

[Geotools-devel] Build failed in Hudson: geotools-trunk #1565

2009-05-10 Thread Hudson
See http://hudson.opengeo.org/hudson/job/geotools-trunk/1565/changes Changes: [jive] Do not use unqiue version numbers for the SNAPSHOTs -- [...truncated 11406 lines...] [INFO] [jar-collector:collect {execution: default}] [INFO] [install:install] [INFO] In

[Geotools-devel] Build failed in Hudson: geotools-trunk #1564

2009-05-10 Thread Hudson
See http://hudson.opengeo.org/hudson/job/geotools-trunk/1564/ -- [...truncated 1696 lines...] Tests run: 13, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.226 sec Running org.geotools.filter.text.ecql.ECQLNullPredicateTest Tests run: 4, Failures: 0, Er

Re: [Geotools-devel] Build failed in Hudson: geotools-trunk #1563

2009-05-10 Thread Jody Garnett
The failure looks to be based on over loading the osgeo repository again (the jar indicated exists) Error transferring file org.opengis:geoapi-pending:jar:2.3-SNAPSHOT from the specified remote repositories: central (http://repo1.maven.org/maven2), osgeo (http://download.osgeo.org/webdav/geoto

[Geotools-devel] Build failed in Hudson: geotools-trunk #1563

2009-05-10 Thread Hudson
See http://hudson.opengeo.org/hudson/job/geotools-trunk/1563/changes Changes: [ang05a] Create app-schema tests artifact so they can be referred by Geoserver app-schema tests. -- [...truncated 1720 lines...] Running org.geotools.filter.text.cql2.CQLGeoOper

[Geotools-devel] udig libs refresh.xml OSGEO repository cleanups

2009-05-10 Thread Jody Garnett
Good afternoon: I have been catching our net.refractions.udig.libs refresh.xml script up with the latest. The latest breakdown of maven repository organizations # used for stable releases updateSnapshotURL=http://download.osgeo.org/webdav/geotools # used for snapshot releases updateStableURL

Re: [Geotools-devel] OSGEO repository, more cleanups

2009-05-10 Thread Jody Garnett
This may be something for Justin - not sure. I tried switching over to the opengeo repository today (to grab snapshot releases) and was unable to obtain 2.6-SNAPSHOTs for: - gt-gml -gt-mif - gt-mysql - gt-oracle-spatial - gt-process - gt-wps And our old friend without source code: - spatialdbbox-

Re: [Geotools-devel] Formal Nomination: Michael Bedward

2009-05-10 Thread Michael Bedward
Thanks very much for your support everyone - much appreciated. > I expect the first couple of things to pay attention to are: > - the remaining nominations (PMC will need to vote on any nominations > that are accepted). > - Andrea is composing a module cleanup page; that will eventually turn > to

Re: [Geotools-devel] ContentDataStore events and jdbc-ng

2009-05-10 Thread Jody Garnett
Thanks Justin: I have updated the developers guide. Thank you for accepting this nomination Michael. Jody On Mon, May 11, 2009 at 9:33 AM, Jody Garnett wrote: > I was going to wait for the methods to settle down first; for example > the batch feature event methods are in the wrong spot; I need

Re: [Geotools-devel] Formal Nomination: Michael Bedward

2009-05-10 Thread Jody Garnett
It looks like I am confusing gmail today ... let me try again. Thanks Justin - that is all the votes in then. Thank you for accepting the nomination Michael - I have added your name to the developers guide. I expect the first couple of things to pay attention to are: - the remaining nominations

Re: [Geotools-devel] Formal Nomination: Michael Bedward

2009-05-10 Thread Justin Deoliveira
Sorry for the late vote. Definitely a +1. -Justin Jody Garnett wrote: > +1 > > And I think that is 75%. > > On Sat, May 9, 2009 at 12:22 AM, Ian Turton wrote: >> >> On Fri, May 8, 2009 at 8:49 AM, Michael Bedward >> wrote: >>> Thanks Jody, for the clarification and for the kinds words. Given

Re: [Geotools-devel] jdbc-ng event patch now ready

2009-05-10 Thread Jody Garnett
On Mon, May 11, 2009 at 10:41 AM, Justin Deoliveira wrote: > I am a little confused... the patch does not seem to touch anything related > to id handling. Am I missing something. Yes; the part where I am stuck on how to patch insert feature writer. > As for null being the placeholder feature id,

Re: [Geotools-devel] jdbc-ng event patch now ready

2009-05-10 Thread Justin Deoliveira
I am a little confused... the patch does not seem to touch anything related to id handling. Am I missing something. As for null being the placeholder feature id, are you talking about in REsultSetFeature? What do you want to change it to? Id generation is a bit tricky. The way it is done now is

Re: [Geotools-devel] Nomination process

2009-05-10 Thread Jody Garnett
I would be content with 7 as well; would you like to make a motion and we can update the developers guide? On Mon, May 11, 2009 at 8:59 AM, Justin Deoliveira wrote: > >> >> I personally find the GeoServer directions sensible, voting is >> easier when the number of people is small, but not too sma

Re: [Geotools-devel] ContentDataStore events and jdbc-ng

2009-05-10 Thread Jody Garnett
I was going to wait for the methods to settle down first; for example the batch feature event methods are in the wrong spot; I need to move them to a transaction state associated with the connection. I will get the javadocs in order shortly; ie after I know we have the problem licked. I will also

Re: [Geotools-devel] jdbc-ng event patch now ready

2009-05-10 Thread Jody Garnett
Justin I need a code review on the feature id handling; and the handling of commit prior to this patch being acceptable to me: 1) BatchFeatureEvent should be notified of any change to FeatureId; right now you are using null as an ID placeholder and I would like to change that. I also wanted to ask

Re: [Geotools-devel] Modules cleanup wiki page created, please review

2009-05-10 Thread Justin Deoliveira
Nice work. I made a few minor comments for xml-po, some of the demo modules, postgis module, and the emf demo modules. I am also +1 on all the ones you flagged as remove. I am looking forward to a smaller code base, it is long overdue. -Justin Andrea Aime wrote: > Hi all, > I've created a wiki

Re: [Geotools-devel] Modules cleanup wiki page created, please review

2009-05-10 Thread Rob Atkinson
re geometryless module - it is in use by people using the 1.6 complex schema support. I dont plan to do anything else with it: I hope to retire it completely by using the app-schema support for complex data types and the ability to handle SQL dialects in the jdbc-ng stack. It has the status of a le

Re: [Geotools-devel] Nomination process

2009-05-10 Thread Justin Deoliveira
> > I personally find the GeoServer directions sensible, voting is > easier when the number of people is small, but not too small. > For a project with 90+ modules the current 5 members is certainly > a bit small, hard to get enough people to cover all point of views, > yet going over 7 may make

Re: [Geotools-devel] jdbc-ng event patch now ready

2009-05-10 Thread Justin Deoliveira
Cool, just reviewed and made some comments on the jira issue. -Justin Jody Garnett wrote: > Hi Justin: > > There is a patch ready for your review: > - http://jira.codehaus.org/browse/GEOT-2327 > > There is some additional work that could be done if interested; right > now a batch feature event

Re: [Geotools-devel] ContentDataStore events and jdbc-ng

2009-05-10 Thread Justin Deoliveira
It would also be nice if you added javadocs for all the methods added, not just some. I have been striving to maintain consistency in the Content* classes, as well as in the JDBC subclasses, that is a pattern I like to keep on following :). For instance fireFeatureUpdated() has no jabadoc, but

Re: [Geotools-devel] ContentDataStore events and jdbc-ng

2009-05-10 Thread Justin Deoliveira
Looks like you have been busy Jody. One thing I ask, can you separate "cleanup" commits. It makes commit logs hard to follow when they are riddled with formatting changes. I am all for cleaning up the formatting, just prefer they be done in a separate commit. Jody Garnett wrote: > Turns out the

Re: [Geotools-devel] Modules cleanup wiki page created, please review

2009-05-10 Thread Andrea Aime
Christian Müller ha scritto: > Andrea, should we edit the wiki page directly ?. I guess so. Just add a comment with your initials in the modules you have an opinion for. > As an example, I agreed with David Adler on the new db2-ng module, so > there will certainly be not further development and

Re: [Geotools-devel] Modules cleanup wiki page created, please review

2009-05-10 Thread Christian Müller
Andrea, should we edit the wiki page directly ?. As an example, I agreed with David Adler on the new db2-ng module, so there will certainly be not further development and no support for the old db2 module. We can and should remove it or move it for some time to unsupported and remove it in th

[Geotools-devel] Modules cleanup wiki page created, please review

2009-05-10 Thread Andrea Aime
Hi all, I've created a wiki page to organise the module maintainerships reassignment and cleanup, here is it: http://docs.codehaus.org/display/GEOTOOLS/Modules+cleanup Ideally, I would like to find a maintainer for each module, and at least an expression of interest for unsupported modules by some

[Geotools-devel] [jira] Created: (GEOT-2476) Updating geometry fails to supply SRID; resulting in exception writing geometry column

2009-05-10 Thread Jody Garnett (JIRA)
Updating geometry fails to supply SRID; resulting in exception writing geometry column -- Key: GEOT-2476 URL: http://jira.codehaus.org/browse/GEOT-2476 Project: GeoTo

[Geotools-devel] jdbc-ng event patch now ready

2009-05-10 Thread Jody Garnett
Hi Justin: There is a patch ready for your review: - http://jira.codehaus.org/browse/GEOT-2327 There is some additional work that could be done if interested; right now a batch feature event is sent out on commit or rollback - one for each feature type modified. This is a bit over kill; a single

Re: [Geotools-devel] ContentDataStore events and jdbc-ng

2009-05-10 Thread Jody Garnett
Sorry for the confusion Andrea; I am confusing h2/hsql and hibernate; I was thinking I should switch to test cases for H2 in order to test event notification. Jody On Sun, May 10, 2009 at 8:36 PM, Jody Garnett wrote: >>> notification code; I think I am being held up over some of the tests >>> not

Re: [Geotools-devel] ContentDataStore events and jdbc-ng

2009-05-10 Thread Andrea Aime
Jody Garnett ha scritto: >>> notification code; I think I am being held up over some of the tests >>> not working for postgis; perhaps I will switch to hibernate. >> This last sentence need some clarification... makes no sense to me? ;-) > > When I last checked the jdbcng postgis was avoiding some

Re: [Geotools-devel] ContentDataStore events and jdbc-ng

2009-05-10 Thread Jody Garnett
>> notification code; I think I am being held up over some of the tests >> not working for postgis; perhaps I will switch to hibernate. > > This last sentence need some clarification... makes no sense to me? ;-) When I last checked the jdbcng postgis was avoiding some of the tests it could not pas

Re: [Geotools-devel] ContentDataStore events and jdbc-ng

2009-05-10 Thread Andrea Aime
Jody Garnett ha scritto: > Turns out the JDBCFeatureReader copy constructor just missed the > feature source field; all tests now pass with the event notification > code in. > > I am going to go on to write test cases covering the event > notification code; I think I am being held up over some of

Re: [Geotools-devel] Vote: graduating the charts module into supported land

2009-05-10 Thread Andrea Aime
Christian Müller ha scritto: > Another quick question about advantages / disadvantages about Sphinx. Is > there a possibility for a spell check ?. Right. There is no IDE like spell check, but you can: - use the one built into the editor of choice. For example to edit Sphinx files I used Gedit