Unterhaltung mit #deegree

(14:00:26) Das Thema für #deegree ist: Welcome to deegree, an OSGeo project. Visit the main project page at http://deegree.org and our wiki at http://wiki.deegree.org with lots of extra info. Check out a running system at http://demo.deegree.org and follow us on twitter @deegree_org.
(14:00:26) Das Thema für #deegree gesetzt von jmays um 15:03:31 auf 10.11.2010
(14:01:01) StephanR [~StephanR@5.146.117.85] hat den Raum betreten.
(14:01:11) StephanR: Hello
(14:01:15) dbretschneider [~Thunderbi@static-87-79-89-38.netcologne.de] hat den Raum betreten.
(14:02:00) mrsnyder: hi
(14:02:08) mrsnyder: everybody on board?
(14:02:22) tfr42: What about Reijer?
(14:02:22) mrsnyder: Reijer is not coming, so let's go.
(14:02:37) mrsnyder: Welcome
(14:02:50) mrsnyder: meeting page here: http://wiki.deegree.org/deegreeWiki/TmcMeeting/TmcMeeting20141111
(14:03:00) mrsnyder: anything to add from your side?
(14:03:48) mrsnyder: no?
(14:03:48) mrsnyder: good
(14:03:57) tfr42: No, please proceed
(14:04:10) mrsnyder: before we start with the pulls, there's one inquiry
(14:04:25) mrsnyder: i forwarded the email a few minutes ok
(14:04:34) mrsnyder: can you please check it
(14:05:38) tfr42: check done
(14:05:43) mrsnyder: good
(14:05:48) StephanR: me too
(14:05:53) mrsnyder: any suggestions for handling this?
(14:06:36) tfr42: He can ask us via the mailing list
(14:06:52) tfr42: Niels did some work with WPS and huge files recently
(14:06:58) lg [1f9605c4@gateway/web/freenode/ip.31.150.5.196] hat den Raum betreten.
(14:07:07) tfr42: I think we can give some input
(14:07:14) mrsnyder: good.
(14:07:15) tfr42: wrt WPS and JVM tuning
(14:07:30) tfr42: but any input is appreciated
(14:07:50) mrsnyder: that's great. i will tell him to email his request to the ml and tell him about your work
(14:08:07) mrsnyder: now for the pull requests
(14:08:23) mrsnyder: Only two requests for 3.3.13:
(14:08:24) mrsnyder: https://github.com/deegree/deegree3/pulls?q=is%3Aopen+is%3Apr+milestone%3A3.3.13
(14:08:48) mrsnyder: https://github.com/deegree/deegree3/pull/215
(14:09:51) mrsnyder: if it breaks compliance, it is not ready...
(14:10:03) mrsnyder: but how can we help him?
(14:10:59) mrsnyder: from point this is: -1
(14:11:07) mrsnyder: from me, i meant
(14:11:21) tfr42: put the issue back to "needs discussion"
(14:11:32) mrsnyder: ok
(14:11:33) StephanR: it is breaking this is a problem: -1
(14:11:38) tfr42: and can give it a -1 only at the moment
(14:11:41) mrsnyder: ok
(14:11:46) tfr42: skip
(14:12:10) mrsnyder: Next:
(14:12:14) mrsnyder: https://github.com/deegree/deegree3/pull/327
(14:12:32) mrsnyder: we discussed that already, but this seems to be lost
(14:12:43) mrsnyder: last time, you asked me for an integration test
(14:14:16) mrsnyder: so we postpone it again?
(14:14:56) tfr42: Well, what is the opinion of the others?
(14:15:18) mrsnyder: i would be fine with merging it :-D
(14:15:28) StephanR: personally i like the feature but wirteing a testcase that is checks the complete szenario will be much work
(14:15:31) tfr42: I havn't tested myself so far
(14:15:54) mrsnyder: stephanR: it turned out to be not too easy
(14:16:06) mrsnyder: i did test it, so did the customer
(14:16:17) mrsnyder: from me: +1
(14:16:29) tfr42: and how did you do it? any tools?
(14:16:38) tfr42: sorry, for asking again
(14:16:38) mrsnyder: no, manually
(14:16:41) tfr42: ok
(14:16:59) mrsnyder: don't get me wrong: i would favor an automated test.
(14:17:04) tfr42: did you observe the runtime behaviour over time
(14:17:06) tfr42: ?
(14:17:08) mrsnyder: yes
(14:17:41) mrsnyder: we basically did a few tests with different time-outs and checked that the interruption cleaned up everything
(14:17:43) tfr42: under load (20+ threads in parallel) too?
(14:17:46) mrsnyder: no
(14:18:01) tfr42: any concurrency checks?
(14:18:02) mrsnyder: but don't believe this is an issue here
(14:18:04) mrsnyder: no
(14:18:31) mrsnyder: moment
(14:18:33) mrsnyder: yes,
(14:18:53) mrsnyder: we did test, that the correct thread is interrupted
(14:19:01) tfr42: ok
(14:19:05) tfr42: that's good
(14:19:19) tfr42: well, then let go +1
(14:19:26) StephanR: than i would also say: +1
(14:19:56) mrsnyder: Merged
(14:20:09) mrsnyder: 3.4 variant: https://github.com/deegree/deegree3/pull/326
(14:20:12) mrsnyder: +1
(14:20:21) mrsnyder: i will merge it as well, yes?
(14:20:37) tfr42: +1
(14:20:39) tfr42: sure
(14:20:42) StephanR: +1
(14:20:55) tfr42: changed the labels too
(14:20:58) mrsnyder: thanks
(14:21:00) mrsnyder: merged
(14:21:06) mrsnyder: 3.4 pulls now
(14:21:07) mrsnyder: https://github.com/deegree/deegree3/pulls?q=is%3Aopen+is%3Apr+milestone%3A3.4
(14:21:26) mrsnyder: which shall we consider?
(14:21:34) StephanR: for the 3.4 codebase i woul like to start wiht #431 (codesprint)
(14:21:43) tfr42: only those without "needs discussion"
(14:21:47) mrsnyder: hm
(14:21:52) tfr42: put them at the end
(14:22:06) mrsnyder: ok, then let's start with #431
(14:22:10) tfr42: #431 is fine
(14:22:26) tfr42: cause it got a OK from travis
(14:22:31) mrsnyder: https://github.com/deegree/deegree3/pull/431
(14:22:59) mrsnyder: +1
(14:23:25) StephanR: +1
(14:23:26) tfr42: excent stuff. hope that solves the travis ci errors
(14:23:29) tfr42: +1
(14:24:22) mrsnyder: Oh no.
(14:24:29) mrsnyder: It cannot be merged automatically.
(14:25:01) mrsnyder: What shall we do?
(14:25:19) tfr42: but how did travis did the build then?
(14:25:22) mrsnyder: Shall we decide to merge and Stephan rebases it after the meeting?
(14:25:41) mrsnyder: probably #326 (which i just merged) lead to a conflict
(14:25:52) StephanR: the build was made before the last merge was made
(14:25:57) tfr42: I see, that's an explanation
(14:26:27) mrsnyder: Stephan, can you rebase it soon after the meeting?
(14:26:28) tfr42: can we revert #326
(14:26:30) tfr42: ?
(14:26:42) tfr42: and then re-try #431 again?
(14:27:05) mrsnyder: sure, but then we have inconsistency between 3.3 and 3.4 functionality
(14:27:08) mrsnyder: also a no-go
(14:27:12) StephanR: i could do this, but i don't know if more problem come up if we merge the current open pulls
(14:27:23) mrsnyder: right
(14:27:40) mrsnyder: i can only offer to assist you
(14:27:50) tfr42: well, what about to stop here and now
(14:28:14) tfr42: change to skype and do the merge in time?
(14:28:23) mrsnyder: fine by me
(14:28:31) tfr42: as soon as we are done, we continue
(14:28:55) tfr42: @StepanR: What about you?
(14:29:12) StephanR: i would have no problem with it
(14:29:17) mrsnyder: good
(14:29:21) tfr42: perfect
(14:29:32) tfr42: Than let's have a break here
(14:29:37) tfr42: Sorry, guys
(14:29:47) tfr42: and switch to Skype
(14:29:57) tfr42: for voice and screen sharing
(14:30:01) tfr42: OK?
(14:30:26) StephanR: ok
(14:30:28) mrsnyder: ok
(14:30:53) tfr42: I will call you wait a moment
(14:39:07) tfr42: Dear listener, please stay online.... we are back soon....
(14:58:33) mrsnyder: #431 rebased by Stephan
(14:58:42) mrsnyder: Merge?
(14:58:45) tfr42: +1
(14:58:45) mrsnyder: +1
(14:58:46) StephanR: +1
(14:59:20) mrsnyder: #366 is obsolete.
(14:59:22) mrsnyder: Delete?
(14:59:42) StephanR: yes, please
(14:59:48) tfr42: +1
(14:59:52) StephanR: +1
(15:17:42) mrsnyder: https://github.com/deegree/deegree3/pull/404
(15:17:52) mrsnyder: Please give your votes:
(15:19:17) StephanR: -1 (as i do not see the corresponding documentation for the schema change in security.xsd)
(15:19:25) mrsnyder: To me, this appears to be too specific. I would prefer a solution for generally dealing with HTTP headers -- or even initializing the Apache HttpClient object.
(15:19:28) mrsnyder: -1
(15:19:32) tfr42: -1 due to missing doc and still a more generic approach is required
(15:25:44) mrsnyder: https://github.com/deegree/deegree3/pull/405
(15:25:50) mrsnyder: Please give your votes:
(15:28:10) tfr42: -1 same as for #404
(15:28:51) StephanR: -1
(15:29:45) mrsnyder: https://github.com/deegree/deegree3/pull/411/files
(15:29:49) mrsnyder: https://github.com/deegree/deegree3/pull/411
(15:30:38) mrsnyder: Please provide more expressive titles.
(15:30:48) mrsnyder: Changed to: Enable parsing of metadata records with empty dates
(15:30:58) mrsnyder: Please give your votes:
(15:31:13) tfr42: +1
(15:31:21) StephanR: +1
(15:31:31) mrsnyder: looks very reasonable: +1
(15:32:19) mrsnyder: merged
(15:35:30) mrsnyder: https://github.com/deegree/deegree3/pull/415
(15:35:47) mrsnyder: Please give your votes:
(15:35:55) mrsnyder: +1
(15:36:04) StephanR: +1
(15:36:10) tfr42: +1
(15:36:24) mrsnyder: merged
(15:38:37) mrsnyder: https://github.com/deegree/deegree3/pull/429
(15:38:48) mrsnyder: Please give your votes:
(15:39:01) mrsnyder: 0
(15:39:28) StephanR: +1
(15:40:01) tfr42: +1
(15:40:15) mrsnyder: merged
(15:42:22) mrsnyder: https://github.com/deegree/deegree3/pull/432
(15:42:27) mrsnyder: Please give your votes:
(15:42:32) StephanR: +1
(15:42:36) mrsnyder: +1
(15:42:47) tfr42: +1 perfect!!!!
(15:46:56) mrsnyder: https://github.com/deegree/deegree3/pull/437
(15:47:36) StephanR: +1
(15:47:42) tfr42: +1
(15:47:47) mrsnyder: +1
(15:47:56) mrsnyder: merged
(15:48:52) mrsnyder: https://github.com/deegree/deegree3/pull/438
(15:50:16) mrsnyder: +1
(15:50:20) StephanR: +1
(15:50:23) tfr42: +1
(15:50:32) mrsnyder: merged
(15:51:00) mrsnyder: https://github.com/deegree/deegree3/pull/438
(15:51:05) mrsnyder: https://github.com/deegree/deegree3/pull/439
(15:51:06) tfr42: +1
(15:51:07) StephanR: +1
(15:51:08) mrsnyder: +1
(15:51:17) mrsnyder: merged
(16:03:55) mrsnyder: https://github.com/deegree/deegree3/pull/447
(16:05:22) mrsnyder: +1
(16:07:09) StephanR: +1
(16:07:32) tfr42: +1
(16:07:39) mrsnyder: merged
(16:11:22) mrsnyder: https://github.com/deegree/deegree3/pull/448
(16:12:51) mrsnyder: +1
(16:13:02) tfr42: +1
(16:13:33) StephanR: +1 (but wait for test)
(16:18:20) mrsnyder: https://github.com/deegree/deegree3/pull/449
(16:18:22) mrsnyder: +1
(16:18:24) tfr42: +1
(16:18:25) StephanR: +1
(16:18:31) mrsnyder: waiting for travis
(16:23:01) mrsnyder: https://github.com/deegree/deegree3/pull/450
(16:23:22) mrsnyder: +1
(16:23:28) tfr42: +1
(16:23:56) StephanR: +1 (but wait for travis)