You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jena.apache.org by "Andy Seaborne (JIRA)" <ji...@apache.org> on 2011/03/24 17:16:05 UTC

[jira] [Created] (JENA-56) Add timeout processing to QueryExecutionHTTP

Add timeout processing to QueryExecutionHTTP
--------------------------------------------

                 Key: JENA-56
                 URL: https://issues.apache.org/jira/browse/JENA-56
             Project: Jena
          Issue Type: Sub-task
          Components: ARQ
            Reporter: Andy Seaborne




--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Andy Seaborne (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13175383#comment-13175383 ] 

Andy Seaborne commented on JENA-56:
-----------------------------------

Time to first answer; time to final answer (overall timeout).

It's quite usage to have a shorter timeout1 because once answers do start flowing, it's reasonable to want to get all of them over a longer period rather than truncate.

                
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Rob Vesse (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13175377#comment-13175377 ] 

Rob Vesse commented on JENA-56:
-------------------------------

Was going to try and do the patch for this today.  Does anyone happen to know what the intended function of the setTimeout(long timeout1, long timeout2) and setTimeout(long timeout1, TimeUnit timeunit1, long timeout2, TimeUnit timeunit2) methods are?

I don't see why there would need to be two timeouts here?
                
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Rob Vesse (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13175394#comment-13175394 ] 

Rob Vesse commented on JENA-56:
-------------------------------

I assume the timeout to final answer would probably be in the actual ARQ QueryIterator level rather than at the HTTP level as my patch does things currently?
                
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>         Attachments: QueryEngineHTTP_TimeoutsPatch.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Closed] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Andy Seaborne (Closed) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andy Seaborne closed JENA-56.
-----------------------------

    
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>            Assignee: Andy Seaborne
>         Attachments: QueryEngineHTTP_TimeoutsPatch.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Andy Seaborne (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andy Seaborne updated JENA-56:
------------------------------

    Summary: Add timeout processing to QueryEngineHTTP  (was: Add timeout processing to QueryExecutionHTTP)

> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>


--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Resolved] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Andy Seaborne (Resolved) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andy Seaborne resolved JENA-56.
-------------------------------

    Resolution: Duplicate
      Assignee: Andy Seaborne
    
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>            Assignee: Andy Seaborne
>         Attachments: QueryEngineHTTP_TimeoutsPatch.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Rob Vesse (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13175393#comment-13175393 ] 

Rob Vesse commented on JENA-56:
-------------------------------

Ok, the patch I've just attached does it completely differently.  It supports timeout for making the connection and timeout for reads, the latter is equivalent to timeout to get first answer because it relates to how long the code waits for data to start coming back from the HTTP response.

Also please note that I had to semi-manually create the patch file because I haven't found a nice OS X subversion client that will do it for me yet so had to do a svn diff which included differences with .project and .settings file which I then manually removed from the generated file.
                
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>         Attachments: QueryEngineHTTP_TimeoutsPatch.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Resolved] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Andy Seaborne (Resolved) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andy Seaborne resolved JENA-56.
-------------------------------

    Resolution: Duplicate

Subsumed by JENA-210
                
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>         Attachments: QueryEngineHTTP_TimeoutsPatch.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Rob Vesse (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rob Vesse updated JENA-56:
--------------------------

    Attachment: QueryEngineHTTP_TimeoutsPatch.patch

Patch provides support for setting connection and read timeouts on QueryEngineHTTP which are passed through to HttpQuery and applied to the actual HttpURLConnection when the HTTP request is made
                
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>         Attachments: QueryEngineHTTP_TimeoutsPatch.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Reopened] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Andy Seaborne (Reopened) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andy Seaborne reopened JENA-56:
-------------------------------

    
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>         Attachments: QueryEngineHTTP_TimeoutsPatch.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Rob Vesse (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13175379#comment-13175379 ] 

Rob Vesse commented on JENA-56:
-------------------------------

Ok, having delved further into the low level code I think I can see why you might want two timeouts - one for HTTP connection and one for HTTP read.

In that case the question becomes which timeout does the simpler setTimeout(long timeout) and setTimeout(long timeout, TimeUnit timeoutUnits) method set - the connect or the read?
                
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Rob Vesse (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13171296#comment-13171296 ] 

Rob Vesse commented on JENA-56:
-------------------------------

I actually have code for this which I am using in an internal benchmarking tool which we'll be open sourcing at some stage anyway, will convert this code into a patch and contribute when I have chance.
                
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (JENA-56) Add timeout processing to QueryEngineHTTP

Posted by "Rob Vesse (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/JENA-56?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13204892#comment-13204892 ] 

Rob Vesse commented on JENA-56:
-------------------------------

The patch submitted as JENA-210 wraps up this patch and some other enhancements to QueryEngineHttp
                
> Add timeout processing to QueryEngineHTTP
> -----------------------------------------
>
>                 Key: JENA-56
>                 URL: https://issues.apache.org/jira/browse/JENA-56
>             Project: Jena
>          Issue Type: Sub-task
>          Components: ARQ
>            Reporter: Andy Seaborne
>         Attachments: QueryEngineHTTP_TimeoutsPatch.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira