Please test it and include in the next version if is possible: Modifications in rest-http module (Added JSF,jndi for jdbc/ignite and tunning for threads of jetty embedded)

classic Classic list List threaded Threaded
4 messages Options
mastercracker3d mastercracker3d
Reply | Threaded
Open this post in threaded view
|

Please test it and include in the next version if is possible: Modifications in rest-http module (Added JSF,jndi for jdbc/ignite and tunning for threads of jetty embedded)

Hi community, I modified the Java Classes: GridJettyRestHandler and GridJettyRestProtocol It's including support for JSF WebPages, auto load JNDI for JDBC/Ignite, Tunning for Thread of Jetty Embedded. Please remove old libraries of Jetty. The list of depencies are the next: commons-dbcp2-2.7.0.jar commons-lang-2.6.jar commons-pool2-2.8.0.jar el-ri-1.0.jar ignite-rest-http-2.7.6.jar must be Modified with the classes GridJettyRestHandler and GridJettyRestProtocol) ignite-web-2.7.6.jar jackson-annotations-2.9.6.jar jackson-core-2.9.6.jar jackson-databind-2.9.6.jar javax.el-api-3.0.1-b06.jar javax.servlet-api-4.0.1.jar jboss-jsf-api_2.2_spec-2.2.14.jar jetty-continuation-9.4.22.v20191022.jar jetty-http-9.4.22.v20191022.jar jetty-io-9.4.22.v20191022.jar jetty-jndi-9.4.22.v20191022.jar jetty-plus-9.4.22.v20191022.jar jetty-security-9.4.22.v20191022.jar jetty-server-9.4.22.v20191022.jar jetty-servlet-9.4.22.v20191022.jar jetty-util-9.4.22.v20191022.jar jetty-webapp-9.4.22.v20191022.jar jetty-xml-9.4.22.v20191022.jar jsf-api-2.2.19.jar jsf-impl-2.2.19.jar jsp-api-2.2.1-b03.jar log4j-1.2.17.jar slf4j-api-1.7.7.jar slf4j-log4j12-1.7.7.jar tomcat-servlet-api-9.0.10.jar Inside of the web.xml in the webpage is need add the next lines: Ignite DataSource Reference jdbc/ignite javax.sql.DataSource Container com.sun.faces.config.ConfigureListener Call to Datasource from java: (DataSource) initContext.lookup("java:comp/env/jdbc/ignite"); GridJettyRestHandler.java GridJettyRestProtocol.java

Sent from the Apache Ignite Users mailing list archive at Nabble.com.
Alexey Kuznetsov-2 Alexey Kuznetsov-2
Reply | Threaded
Open this post in threaded view
|

Re: Please test it and include in the next version if is possible: Modifications in rest-http module (Added JSF,jndi for jdbc/ignite and tunning for threads of jetty embedded)

Hi, mastercracker3d!

The right way to change code is the following:
1. Create account in Apache Ignite JIRA https://issues.apache.org/jira/projects/IGNITE
2. Create issue in JIRA.
3. Prepare pull request.
5. After successful review you PR may be merged to master.

Hope this helps!

On Sun, Dec 29, 2019 at 12:38 AM mastercracker3d <[hidden email]> wrote:
Hi community, I modified the Java Classes: GridJettyRestHandler and GridJettyRestProtocol It's including support for JSF WebPages, auto load JNDI for JDBC/Ignite, Tunning for Thread of Jetty Embedded. Please remove old libraries of Jetty. The list of depencies are the next: commons-dbcp2-2.7.0.jar commons-lang-2.6.jar commons-pool2-2.8.0.jar el-ri-1.0.jar ignite-rest-http-2.7.6.jar must be Modified with the classes GridJettyRestHandler and GridJettyRestProtocol) ignite-web-2.7.6.jar jackson-annotations-2.9.6.jar jackson-core-2.9.6.jar jackson-databind-2.9.6.jar javax.el-api-3.0.1-b06.jar javax.servlet-api-4.0.1.jar jboss-jsf-api_2.2_spec-2.2.14.jar jetty-continuation-9.4.22.v20191022.jar jetty-http-9.4.22.v20191022.jar jetty-io-9.4.22.v20191022.jar jetty-jndi-9.4.22.v20191022.jar jetty-plus-9.4.22.v20191022.jar jetty-security-9.4.22.v20191022.jar jetty-server-9.4.22.v20191022.jar jetty-servlet-9.4.22.v20191022.jar jetty-util-9.4.22.v20191022.jar jetty-webapp-9.4.22.v20191022.jar jetty-xml-9.4.22.v20191022.jar jsf-api-2.2.19.jar jsf-impl-2.2.19.jar jsp-api-2.2.1-b03.jar log4j-1.2.17.jar slf4j-api-1.7.7.jar slf4j-log4j12-1.7.7.jar tomcat-servlet-api-9.0.10.jar Inside of the web.xml in the webpage is need add the next lines: Ignite DataSource Reference jdbc/ignite javax.sql.DataSource Container com.sun.faces.config.ConfigureListener Call to Datasource from java: (DataSource) initContext.lookup("java:comp/env/jdbc/ignite"); GridJettyRestHandler.java GridJettyRestProtocol.java

Sent from the Apache Ignite Users mailing list archive at Nabble.com.


--
Alexey Kuznetsov
mastercracker3d mastercracker3d
Reply | Threaded
Open this post in threaded view
|

Re: Please test it and include in the next version if is possible: Modifications in rest-http module (Added JSF,jndi for jdbc/ignite and tunning for threads of jetty embedded)

Thanks Alexey,
I make other change in the code for support root context for the webpage.
(if is not present, ignite is a context root and ignite has /ignite context)


GridJettyRestHandler.java
<http://apache-ignite-users.70518.x6.nabble.com/file/t1813/GridJettyRestHandler.java>  

GridJettyRestProtocol.java
<http://apache-ignite-users.70518.x6.nabble.com/file/t1813/GridJettyRestProtocol.java>  





--
Sent from: http://apache-ignite-users.70518.x6.nabble.com/
mastercracker3d mastercracker3d
Reply | Threaded
Open this post in threaded view
|

Re: Please test it and include in the next version if is possible: Modifications in rest-http module (Added JSF,jndi for jdbc/ignite and tunning for threads of jetty embedded)

In reply to this post by mastercracker3d
the new internal case is: IGNITE-12518



--
Sent from: http://apache-ignite-users.70518.x6.nabble.com/