> Passing the ball back on this... > Catch. > So I first made a test case for this (both on the auto created test table and > on our problematic one) and couldn't reproduce. > Thanks for making the test cases Gabriel - there goes that idea (although it is strange that neither of us can produce exactly the same error today?). > Once I got udig running again, found all and every "shared vertex" > modification produced an exception, yet it was not very clear why. > Further investigation lead realize the JTS Geometry to SeShape convertion > weren't possible, SeShape complaining about topology validity. But it was not > at the server side whatsoever, just trying to create an SeShape from the > geometry coordinates. > > anyhow, I did fixed a bug, if something went wrong I wasn't rolling back the > transaction and even if FeatureWriter were throwing out an exception the > transaction commited and we got partial results (one feature updated and the > other not). > Okay that explains that part. I wonder if SeShape has different restrictions that JTS Geometry? > But for what we're discussing here, it seems udig is indeed sending invalid > geometries to FeatureStore.modifyFeatures. > I can check again; currently all the udig commands are set to fail if a Geomtry is inValid (and everything has slowed down so much I am sure we are running those checks). > I've improved the message reported by the exception thrown, and tested with > JTS testBuilder a lot of times. Even the smaller modification to a "shared > verex" leads to one of the geometries sent being invalid, as per > Geometry.isValid(). > Okay; the ball really is in my court and I will try and verify when I get near ArcSDE later this morning. > The error reporting from the arcsde plugin now looks like the two examples > bellow, so you can load the WKT on testBuilder and hit "isValid?") > > (btw, sorry for the extension of the message) > No worries; thanks for the detail - it is enough to be helpful. Jody > Exception messages thrown by gt2-arcsde (note the "Topology validation > error..." message at the end: > > org.geotools.data.DataSourceException: Invalid geometry passed to > SDE.CJ_TST_1.SHAPE > Geomerty: MULTIPOLYGON (((369841.326 601228.047, 369849.948 601217.756, > 369855.141 601215.16, 369865.526 601216.025, 369874.756 601216.025, > 369876.827 601216.439, 369881.969 601217.467, 369886.194 601218.876, > 369886.546 601214.69, 369887.112 601207.951, 369880.73 601201.037, 369872.37 > 601192.677, 369866.822 601194.262, 369864.985 601189.855, 369864.618 > 601185.449, 369862.782 601175.534, 369810.23224911804 601169.8853150774, > 369870.494 601145.054, 369876.369 601134.037, 369888.121 601115.676, > 369898.403 601099.518, 369906.849 601083.36, 369918.967 601066.467, 369922.64 > 601055.451, 369920.437 601049.208, 369912.725 601049.208, 369892.894 > 601056.185, 369861.68 601074.179, 369845.155 601080.422, 369824.958 > 601090.337, 369805.127 601102.456, 369790.806 601114.941, 369776.11699999997 > 601127.06, 369754.45 601138.444, 369735.722 601147.257, 369709.649 > 601153.867, 369689.81799999997 601157.172, 369681.739 601154.602, 369669.621 > 601146.156, 369651.994 601139.546, 369636.571 601132.201, 369631.014 > 601129.17, 369620.953 601136.996, 369609.594 601135.411, 369598.765 > 601130.392, 369587.934 601131.713, 369581.859 601136.468, 369580.803 > 601140.694, 369579.746 601147.562, 369575.519 601149.147, 369567.331 > 601151.524, 369565.993 601151.524, 369570.47 601160.477, 369571.939 > 601174.432, 369587.362 601192.059, 369602.051 601214.093, 369604.989 > 601235.392, 369618.944 601245.674, 369642.446 601248.612, 369671.825 > 601242.736, 369710.751 601234.657, 369739.394 601219.2339999999, > 369765.83400000003 601206.014, 369795.947 601203.81, 369818.71499999997 > 601211.155, 369827.528 601221.437, 369836.342 601228.047, 369841.326 > 601228.047)), ((369841.326 601228.047, 369849.948 601217.756, 369855.141 > 601215.16, 369865.526 601216.025, 369874.756 601216.025, 369876.827 > 601216.439, 369881.969 601217.467, 369886.194 601218.876, 369886.546 > 601214.69, 369887.112 601207.951, 369880.73 601201.037, 369872.37 601192.677, > 369866.822 601194.262, 369864.985 601189.855, 369864.618 601185.449, > 369862.782 601175.534, 369810.23224911804 601169.8853150774, 369870.494 > 601145.054, 369876.369 601134.037, 369888.121 601115.676, 369898.403 > 601099.518, 369906.849 601083.36, 369918.967 601066.467, 369922.64 > 601055.451, 369920.437 601049.208, 369912.725 601049.208, 369892.894 > 601056.185, 369861.68 601074.179, 369845.155 601080.422, 369824.958 > 601090.337, 369805.127 601102.456, 369790.806 601114.941, 369776.11699999997 > 601127.06, 369754.45 601138.444, 369735.722 601147.257, 369709.649 > 601153.867, 369689.81799999997 601157.172, 369681.739 601154.602, 369669.621 > 601146.156, 369651.994 601139.546, 369636.571 601132.201, 369631.014 > 601129.17, 369620.953 601136.996, 369609.594 601135.411, 369598.765 > 601130.392, 369587.934 601131.713, 369581.859 601136.468, 369580.803 > 601140.694, 369579.746 601147.562, 369575.519 601149.147, 369567.331 > 601151.524, 369565.993 601151.524, 369570.47 601160.477, 369571.939 > 601174.432, 369587.362 601192.059, 369602.051 601214.093, 369604.989 > 601235.392, 369618.944 601245.674, 369642.446 601248.612, 369671.825 > 601242.736, 369710.751 601234.657, 369739.394 601219.2339999999, > 369765.83400000003 601206.014, 369795.947 601203.81, 369818.71499999997 > 601211.155, 369827.528 601221.437, 369836.342 601228.047, 369841.326 > 601228.047))) > Topology validation error at or near point (369841.326, 601228.047, NaN): > Duplicate Rings > at > org.geotools.arcsde.data.ArcSdeFeatureWriter.setRowValue(ArcSdeFeatureWriter.java:533) > > > org.geotools.data.DataSourceException: Invalid geometry passed to > SDE.CJ_TST_1.SHAPE > Geomerty: MULTIPOLYGON (((370821.562 601710.456, 370828.461 601691.828, > 370827.771 601676.649, 370872.5878623821 601647.9995128451, 370813.973 > 601648.017, 370807.419 601634.908, 370796.03500000003 601628.354, 370783.961 > 601620.075, 370777.752 601609.036, 370776.372 601598.687, 370772.577 > 601588.339, 370761.88300000003 601585.924, 370753.259 601585.2339999999, > 370742.22 601580.75, 370718.418 601589.028, 370698.41000000003 601596.273, > 370693.926 601606.276, 370698.41000000003 601614.211, 370703.929 601616.625, > 370705.654 601621.8, 370703.58400000003 601631.1140000001, 370698.065 > 601639.393, 370696.685 601650.087, 370703.239 601656.296, 370705.654 > 601658.711, 370705.654 601664.92, 370703.58400000003 601674.58, 370705.999 > 601683.548, 370712.20900000003 601687.688, 370718.073 601686.308, 370724.627 > 601684.238, 370733.596 601685.618, 370735.013 601686.326, 370734.591 > 601685.714, 370735.815 601671.837, 370744.386 601675.102, 370746.427 > 601682.041, 370758.263 601682.857, 370762.753 601690.612, 370778.262 > 601710.203, 370790.507 601710.611, 370800.71 601700.407, 370804.384 > 601705.305, 370803.976 601717.55, 370810.098 601723.263, 370810.098 > 601727.521, 370814.663 601723.564, 370821.562 601710.456)), ((370821.562 > 601710.456, 370828.461 601691.828, 370827.771 601676.649, 370872.5878623821 > 601647.9995128451, 370813.973 601648.017, 370807.419 601634.908, > 370796.03500000003 601628.354, 370783.961 601620.075, 370777.752 601609.036, > 370776.372 601598.687, 370772.577 601588.339, 370761.88300000003 601585.924, > 370753.259 601585.2339999999, 370742.22 601580.75, 370718.418 601589.028, > 370698.41000000003 601596.273, 370693.926 601606.276, 370698.41000000003 > 601614.211, 370703.929 601616.625, 370705.654 601621.8, 370703.58400000003 > 601631.1140000001, 370698.065 601639.393, 370696.685 601650.087, 370703.239 > 601656.296, 370705.654 601658.711, 370705.654 601664.92, 370703.58400000003 > 601674.58, 370705.999 601683.548, 370712.20900000003 601687.688, 370718.073 > 601686.308, 370724.627 601684.238, 370733.596 601685.618, 370735.013 > 601686.326, 370734.591 601685.714, 370735.815 601671.837, 370744.386 > 601675.102, 370746.427 601682.041, 370758.263 601682.857, 370762.753 > 601690.612, 370778.262 601710.203, 370790.507 601710.611, 370800.71 > 601700.407, 370804.384 601705.305, 370803.976 601717.55, 370810.098 > 601723.263, 370810.098 601727.521, 370814.663 601723.564, 370821.562 > 601710.456))) > Topology validation error at or near point (370821.562, 601710.456, NaN): > Duplicate Rings > at > org.geotools.arcsde.data.ArcSdeFeatureWriter.setRowValue(ArcSdeFeatureWriter.java:533) > >
------------------------------------------------------------------------- SF.Net email is sponsored by: The Future of Linux Business White Paper from Novell. From the desktop to the data center, Linux is going mainstream. Let it simplify your IT future. http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4 _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
