GeoTools
  1. GeoTools
  2. GEOT-599

WFSFeatureReader thread gets locked

    Details

    • Type: Bug Bug
    • Status: Open Open
    • Priority: Critical Critical
    • Resolution: Unresolved
    • Affects Version/s: 2.1.RC1
    • Fix Version/s: 2.7.6
    • Component/s: wfs plugin, xml
    • Labels:
      None

      Description

      Trying to get a more or less large attribute value from a WFS (a couple megs string), a timeout exception is thrown and WFSFeatureReader::run() seems not to exit gracefully, consuming 100% CPU.
      So I'm going to work against jody's wfs-testing branch and try tu hunt this bug.

      (20:59:15) groldan: hey Jody, I need to fix a bug in WFSDataStore. I'm going to set up a jira task for it. Once I get it fixed, should I commit or attach a patch to the jira issue? (I ask 'cause I got a bit confused about the way of working on 2.1.x, plus David being the module maintainer and seem to be bussy this days, plus I need it fixed quickly)
      (20:59:25) jodygarnett: Um
      (20:59:30) jodygarnett: I probably fixed it already
      (20:59:38) jodygarnett: remember I made wfs-patch
      (20:59:47) jodygarnett: because the wfs datastore maintainer is away on holidays.
      (20:59:51) jodygarnett: (and I needed udig to work)
      (21:00:10) jodygarnett: http://svn.geotools.org/geotools/branches/wfs-testing/
      (21:00:11) groldan: ah... that would be great..
      (21:00:15) groldan: let me see
      (21:00:18) jodygarnett: try that ... it is what udig runs.
      (21:00:25) jodygarnett: Fixes a number of bugs at the datastore level
      (21:00:29) jodygarnett: mostly reprojection related.
      (21:02:12) groldan: mmm... so may be its not the one I'm looking for... the point is that we're dealing with large attribute values (a couple megs strings), so WFSFeaturReader::run gets a timeout exception from which it can't survive... and the thread gets locked (consuming 100%CPU)
      (21:02:27) groldan: do you remember worked on something similar?
      (21:03:05) jodygarnett: no I don't .. interteresting problem. I had a couple goes at make a bug report and was shot down. Good luck with yours
      (21:03:34) jodygarnett: That said when I have a test case things went well.
      (21:03:41) groldan: ok, will try using your patched anyway, and in case I fix it can I commit to your branch?
      (21:04:00) jodygarnett: I wonder where the timeout is occuring ... if it is on the sax side we may be in trouble.
      (21:04:25) jodygarnett: I think there were some hints you can set to control timeout or something.
      (21:04:32) groldan: actually I'm going to debug it right now, but I'm pretty sure it's on FCBuffer
      (21:04:36) jodygarnett: Wish I could be more help.
      (21:05:07) jodygarnett: Cool. There is a known issue where a bunch of buffers hang around yeilding to each other - using 100% CPU.
      (21:05:07) groldan: yeah, but timeouts are not helping too much (well, at least I used a 30secs timeout)
      (21:05:22) groldan: the strange thing is that geoserver returns pretty fast the result
      (21:05:23) jodygarnett: I wonder if that is the problem you are describing?
      (21:05:41) groldan: It could certainly be

        Activity

        Justin Deoliveira made changes -
        Field Original Value New Value
        Fix Version/s 2.1.1 [ 11907 ]
        Fix Version/s 2.1.2 [ 12303 ]
        Cory Horner made changes -
        Fix Version/s 2.1.2 [ 12303 ]
        Hide
        Jesse Eichar added a comment -
        This still valid?
        Show
        Jesse Eichar added a comment - This still valid?
        Jody Garnett made changes -
        Fix Version/s 2.7-M0 [ 15891 ]
        Jody Garnett made changes -
        Fix Version/s 2.7-M1 [ 16521 ]
        Fix Version/s 2.7-M0 [ 15891 ]
        Jody Garnett made changes -
        Fix Version/s 2.7-RC1 [ 16571 ]
        Fix Version/s 2.7-M1 [ 16521 ]
        Jody Garnett made changes -
        Fix Version/s 2.7-M3 [ 16572 ]
        Fix Version/s 2.7-M2 [ 16571 ]
        Justin Deoliveira made changes -
        Fix Version/s 2.7-M4 [ 16726 ]
        Fix Version/s 2.7-M3 [ 16572 ]
        Andrea Aime made changes -
        Fix Version/s 2.7-beta1 [ 16974 ]
        Fix Version/s 2.7-M4 [ 16726 ]
        Justin Deoliveira made changes -
        Fix Version/s 2.7-RC1 [ 17076 ]
        Fix Version/s 2.7-beta1 [ 16974 ]
        Justin Deoliveira made changes -
        Fix Version/s 2.7.0 [ 17109 ]
        Fix Version/s 2.7-RC1 [ 17076 ]
        Justin Deoliveira made changes -
        Fix Version/s 2.7.1 [ 17255 ]
        Fix Version/s 2.7.0 [ 17109 ]
        Justin Deoliveira made changes -
        Fix Version/s 2.7.2 [ 17359 ]
        Fix Version/s 2.7.1 [ 17255 ]
        Jody Garnett made changes -
        Fix Version/s 2.7.3 [ 17442 ]
        Fix Version/s 2.7.2 [ 17359 ]
        Andrea Aime made changes -
        Fix Version/s 2.7.4 [ 17863 ]
        Fix Version/s 2.7.3 [ 17442 ]
        Justin Deoliveira made changes -
        Fix Version/s 2.7.5 [ 18236 ]
        Fix Version/s 2.7.4 [ 17863 ]
        Justin Deoliveira made changes -
        Fix Version/s 2.7.6 [ 18554 ]
        Fix Version/s 2.7.5 [ 18236 ]

          People

          • Assignee:
            Gabriel Roldan
            Reporter:
            Gabriel Roldan
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: