Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Lucene: Java-Dev

[jira] [Updated] (SOLR-3301) Migrate enable/disable Ping from JSP to PingRequestHandler

 

 

Lucene java-dev RSS feed   Index | Next | Previous | View Threaded


jira at apache

Apr 21, 2012, 6:21 AM

Post #1 of 4 (129 views)
Permalink
[jira] [Updated] (SOLR-3301) Migrate enable/disable Ping from JSP to PingRequestHandler

[ https://issues.apache.org/jira/browse/SOLR-3301?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matt Overstreet updated SOLR-3301:
----------------------------------

Attachment: health_check_admin_consolidate.patch

consolidated patch. includes all changes from the other attached patches as requested.

> Migrate enable/disable Ping from JSP to PingRequestHandler
> ----------------------------------------------------------
>
> Key: SOLR-3301
> URL: https://issues.apache.org/jira/browse/SOLR-3301
> Project: Solr
> Issue Type: Improvement
> Components: web gui
> Affects Versions: 4.0
> Reporter: Eric Pugh
> Assignee: Stefan Matheis (steffkes)
> Fix For: 4.0
>
> Attachments: PingRequestHandlerTest.java, health_check_admin_consolidate.patch, healthcheck-ui.png, healthcheckenable.patch, ping_request_handler.patch
>
>
> My colleague @omnifroodle and I noticed that you can't enable/disable pings in 4.0 because action.jsp doesn't run. This patch attaches the functionality to the PingRequestHandler. We debated creating a new HealthcheckRequestHandler as well. We added some basic unit tests.

--
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



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] lucene
For additional commands, e-mail: dev-help [at] lucene


jira at apache

Apr 21, 2012, 11:54 AM

Post #2 of 4 (126 views)
Permalink
[jira] [Updated] (SOLR-3301) Migrate enable/disable Ping from JSP to PingRequestHandler [In reply to]

[ https://issues.apache.org/jira/browse/SOLR-3301?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matt Overstreet updated SOLR-3301:
----------------------------------

Attachment: health_check_admin_consolidate2.patch

Updated to address Stefan's great issue list. disabled healthcheck now returns with a 503 status.

Also, healthcheck status display works correctly if the healthcheck file is commented out at server start (the entire control is grayed out).

Let me know if I need to make any more tweaks!

> Migrate enable/disable Ping from JSP to PingRequestHandler
> ----------------------------------------------------------
>
> Key: SOLR-3301
> URL: https://issues.apache.org/jira/browse/SOLR-3301
> Project: Solr
> Issue Type: Improvement
> Components: web gui
> Affects Versions: 4.0
> Reporter: Eric Pugh
> Assignee: Stefan Matheis (steffkes)
> Fix For: 4.0
>
> Attachments: PingRequestHandlerTest.java, health_check_admin_consolidate.patch, health_check_admin_consolidate2.patch, healthcheck-ui.png, healthcheckenable.patch, ping_request_handler.patch
>
>
> My colleague @omnifroodle and I noticed that you can't enable/disable pings in 4.0 because action.jsp doesn't run. This patch attaches the functionality to the PingRequestHandler. We debated creating a new HealthcheckRequestHandler as well. We added some basic unit tests.

--
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



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] lucene
For additional commands, e-mail: dev-help [at] lucene


jira at apache

Apr 22, 2012, 10:29 AM

Post #3 of 4 (135 views)
Permalink
[jira] [Updated] (SOLR-3301) Migrate enable/disable Ping from JSP to PingRequestHandler [In reply to]

[ https://issues.apache.org/jira/browse/SOLR-3301?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stefan Matheis (steffkes) updated SOLR-3301:
--------------------------------------------

Attachment: SOLR-3301.patch

Thanks Matt, i've changed the following things:

1) Lucene is using two spaces for indentation, i replaced the tabs.

2) Your Patch did not compile:
{code}+ if (healthcheck == null) {
+ throw new SolrException(SolrException.ErrorCode.SERVICE_UNAVAILABLE x,
+ "No healthcheck file defined.");
+ }{code}
was failing with the following error(s):
{code}common.compile-core:
[mkdir] Created dir: /opt/solr-trunk/solr/build/solr-core/classes/java
[javac] Compiling 564 source files to /opt/solr-trunk/solr/build/solr-core/classes/java
[javac] /opt/solr-trunk/solr/core/src/java/org/apache/solr/handler/PingRequestHandler.java:141: ')' expected
[javac] throw new SolrException(SolrException.ErrorCode.SERVICE_UNAVAILABLE x,
[javac] ^
[javac] /opt/solr-trunk/solr/core/src/java/org/apache/solr/handler/PingRequestHandler.java:141: illegal start of expression
[javac] throw new SolrException(SolrException.ErrorCode.SERVICE_UNAVAILABLE x,
[javac] ^
[javac] /opt/solr-trunk/solr/core/src/java/org/apache/solr/handler/PingRequestHandler.java:141: ';' expected
[javac] throw new SolrException(SolrException.ErrorCode.SERVICE_UNAVAILABLE x,
[javac] ^
[javac] /opt/solr-trunk/solr/core/src/java/org/apache/solr/handler/PingRequestHandler.java:142: illegal start of expression
[javac] "No healthcheck file defined.");
[javac] ^
[javac] 4 errors{code}

3) I would not expect a {{BAD_REQUEST}}-Error if Ping is not configured?
{code}+ case STATUS:
+ if( healthcheck == null){
+ SolrException e = new SolrException(SolrException.ErrorCode.BAD_REQUEST, "healthcheck not configured");
+ rsp.setException(e);
+ }{code}
It's not the Clients Fault (which is, what the 4xx-Status-Range is meant for) - i changed this one into a {{SERVICE_UNAVAILABLE}}, which reflects the behavior more correct imho.

Let me know if these are okay for you, then i'll go ahead and commit this one

> Migrate enable/disable Ping from JSP to PingRequestHandler
> ----------------------------------------------------------
>
> Key: SOLR-3301
> URL: https://issues.apache.org/jira/browse/SOLR-3301
> Project: Solr
> Issue Type: Improvement
> Components: web gui
> Affects Versions: 4.0
> Reporter: Eric Pugh
> Assignee: Stefan Matheis (steffkes)
> Fix For: 4.0
>
> Attachments: PingRequestHandlerTest.java, SOLR-3301.patch, health_check_admin_consolidate.patch, health_check_admin_consolidate2.patch, healthcheck-ui.png, healthcheckenable.patch, ping_request_handler.patch
>
>
> My colleague @omnifroodle and I noticed that you can't enable/disable pings in 4.0 because action.jsp doesn't run. This patch attaches the functionality to the PingRequestHandler. We debated creating a new HealthcheckRequestHandler as well. We added some basic unit tests.

--
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



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] lucene
For additional commands, e-mail: dev-help [at] lucene


jira at apache

Apr 24, 2012, 3:45 AM

Post #4 of 4 (120 views)
Permalink
[jira] [Updated] (SOLR-3301) Migrate enable/disable Ping from JSP to PingRequestHandler [In reply to]

[ https://issues.apache.org/jira/browse/SOLR-3301?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stefan Matheis (steffkes) updated SOLR-3301:
--------------------------------------------

Attachment: SOLR-3301-tests.patch

> Migrate enable/disable Ping from JSP to PingRequestHandler
> ----------------------------------------------------------
>
> Key: SOLR-3301
> URL: https://issues.apache.org/jira/browse/SOLR-3301
> Project: Solr
> Issue Type: Improvement
> Components: web gui
> Affects Versions: 4.0
> Reporter: Eric Pugh
> Assignee: Stefan Matheis (steffkes)
> Fix For: 4.0
>
> Attachments: PingRequestHandlerTest.java, SOLR-3301-tests.patch, SOLR-3301.patch, health_check_admin_consolidate.patch, health_check_admin_consolidate2.patch, healthcheck-ui.png, healthcheckenable.patch, ping_request_handler.patch
>
>
> My colleague @omnifroodle and I noticed that you can't enable/disable pings in 4.0 because action.jsp doesn't run. This patch attaches the functionality to the PingRequestHandler. We debated creating a new HealthcheckRequestHandler as well. We added some basic unit tests.

--
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



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] lucene
For additional commands, e-mail: dev-help [at] lucene

Lucene java-dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.