Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow mongodb steps connect to mongodb atlas #150

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

danielams
Copy link

Hi,
I have made some changes in the project pentaho-mongo-utils for allow the mongodb steps connect with the mongodb atlas through the java driver
https://docs.atlas.mongodb.com/driver-connection/#java-driver-example

Please let me know any improvements that I should do in the source code.

Thanks
Daniel

@wingman-pentaho
Copy link
Collaborator

Hello, This is an automated comment from our build system. Since this pull request originated from a user not in the Pentaho organization, we will not proceed with automatic validation of the pull request. However, rest assured that it will be reviewed manually as soon as we get the chance. Thanks for your contribution!

@wingman-pentaho
Copy link
Collaborator

Build Failed

❌ Something went wrong while validating this pull request.

Build Commands

mvn -B -fn -f pom.xml clean install && mvn -B -f pom.xml site
Stdout log
(last 100 lines)
> :heavy_exclamation_mark: 'try' have incorrect indentation level 8, expected level should be 4. at Line: 1,878

> :heavy_exclamation_mark: 'member def type' have incorrect indentation level 12, expected level should be 6. at Line: 1,879

> :heavy_exclamation_mark: 'try' child have incorrect indentation level 12, expected level should be 6. at Line: 1,879

> :heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 1,879 Column: 52

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,879 Column: 68

> :heavy_exclamation_mark: 'member def type' have incorrect indentation level 12, expected level should be 6. at Line: 1,880

> :heavy_exclamation_mark: 'try' child have incorrect indentation level 12, expected level should be 6. at Line: 1,880

> :heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 1,880 Column: 60

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,880 Column: 80

> :heavy_exclamation_mark: 'member def type' have incorrect indentation level 12, expected level should be 6. at Line: 1,881

> :heavy_exclamation_mark: 'try' child have incorrect indentation level 12, expected level should be 6. at Line: 1,881

> :heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 1,881 Column: 55

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,881 Column: 74

> :heavy_exclamation_mark: 'member def type' have incorrect indentation level 12, expected level should be 6. at Line: 1,882

> :heavy_exclamation_mark: 'try' child have incorrect indentation level 12, expected level should be 6. at Line: 1,882

> :heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 1,882 Column: 56

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,882 Column: 76

> :heavy_exclamation_mark: 'member def type' have incorrect indentation level 12, expected level should be 6. at Line: 1,883

> :heavy_exclamation_mark: 'try' child have incorrect indentation level 12, expected level should be 6. at Line: 1,883

> :heavy_exclamation_mark: 'if' have incorrect indentation level 12, expected level should be 6. at Line: 1,884

> :heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 1,884 Column: 17

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,884 Column: 36

> :heavy_exclamation_mark: 'if' child have incorrect indentation level 16, expected level should be 8. at Line: 1,885

> :heavy_exclamation_mark: 'if rcurly' have incorrect indentation level 12, expected level should be 6. at Line: 1,886

> :heavy_exclamation_mark: 'member def type' have incorrect indentation level 12, expected level should be 6. at Line: 1,887

> :heavy_exclamation_mark: 'try' child have incorrect indentation level 12, expected level should be 6. at Line: 1,887

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,889 Column: 127

> :heavy_exclamation_mark: 'if' have incorrect indentation level 12, expected level should be 6. at Line: 1,893

> :heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 1,893 Column: 17

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,893 Column: 44

> :heavy_exclamation_mark: 'if' child have incorrect indentation level 16, expected level should be 8. at Line: 1,894

> :heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 1,894 Column: 37

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,894 Column: 53

> :heavy_exclamation_mark: 'if' child have incorrect indentation level 16, expected level should be 8. at Line: 1,895

> :heavy_exclamation_mark: 'if rcurly' have incorrect indentation level 12, expected level should be 6. at Line: 1,896

> :heavy_exclamation_mark: 'try rcurly' have incorrect indentation level 8, expected level should be 4. at Line: 1,897

> :heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 1,897 Column: 18

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,897 Column: 29

> :heavy_exclamation_mark: 'if' have incorrect indentation level 12, expected level should be 6. at Line: 1,898

> :heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 1,898 Column: 17

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,898 Column: 45

> :heavy_exclamation_mark: 'if' child have incorrect indentation level 16, expected level should be 8. at Line: 1,899

> :heavy_exclamation_mark: 'if rcurly' have incorrect indentation level 12, expected level should be 6. at Line: 1,900

> :heavy_exclamation_mark: 'else' child have incorrect indentation level 16, expected level should be 8. at Line: 1,901

> :heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 1,901 Column: 43

> :heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,901 Column: 86

> :heavy_exclamation_mark: 'else rcurly' have incorrect indentation level 12, expected level should be 6. at Line: 1,902

> :heavy_exclamation_mark: 'catch rcurly' have incorrect indentation level 8, expected level should be 4. at Line: 1,903

> :heavy_exclamation_mark: 'method def' child have incorrect indentation level 8, expected level should be 4. at Line: 1,904

> :heavy_exclamation_mark: 'method def rcurly' have incorrect indentation level 4, expected level should be 2. at Line: 1,905

Stderr log

(last 100 lines)
Cloning into 'pentaho-mongodb-plugin-base-150-33414742-ea09-4e91-875f-973948bf82f9'...
From https://github.com/pentaho/pentaho-mongodb-plugin
 * [new ref]         refs/pull/150/head -> pullRequest
Copyright year is not the current year on src/main/java/org/pentaho/di/trans/steps/mongodboutput/MongoDbOutputDialog.java. Found 2016. Was expecting 2017
Copyright year is not the current year on src/main/java/org/pentaho/di/trans/steps/mongodboutput/MongoDbOutputMeta.java. Found 2016. Was expecting 2017
Copyright year is not the current year on src/main/java/org/pentaho/di/ui/trans/steps/mongodbinput/MongoDbInputDialog.java. Found 2016. Was expecting 2017
Copyright year is not the current year on src/test/java/org/pentaho/di/trans/steps/mongodboutput/MongoDbOutputMetaTest.java. Found 2015. Was expecting 2017
java.lang.Exception: java.lang.RuntimeException: org.eclipse.egit.github.core.client.RequestException: Validation Failed (422): body is too long (maximum is 65536 characters)
	at org.pentaho.build.buddy.bundles.orchestrator.OrchestratorImpl.orchestrate(OrchestratorImpl.java:369)
	at Proxye9f595a1_5eb9_466e_bd7f_57305db87c6e.orchestrate(Unknown Source)
	at org.pentaho.build.buddy.bundles.rest.OrchestratorRestService$1.write(OrchestratorRestService.java:62)
	at org.apache.cxf.jaxrs.provider.BinaryDataProvider.writeTo(BinaryDataProvider.java:172)
	at org.apache.cxf.jaxrs.utils.JAXRSUtils.writeMessageBody(JAXRSUtils.java:1381)
	at org.apache.cxf.jaxrs.interceptor.JAXRSOutInterceptor.serializeMessage(JAXRSOutInterceptor.java:244)
	at org.apache.cxf.jaxrs.interceptor.JAXRSOutInterceptor.processResponse(JAXRSOutInterceptor.java:120)
	at org.apache.cxf.jaxrs.interceptor.JAXRSOutInterceptor.handleMessage(JAXRSOutInterceptor.java:83)
	at org.apache.cxf.phase.PhaseInterceptorChain.doIntercept(PhaseInterceptorChain.java:308)
	at org.apache.cxf.interceptor.OutgoingChainInterceptor.handleMessage(OutgoingChainInterceptor.java:83)
	at org.apache.cxf.phase.PhaseInterceptorChain.doIntercept(PhaseInterceptorChain.java:308)
	at org.apache.cxf.transport.ChainInitiationObserver.onMessage(ChainInitiationObserver.java:121)
	at org.apache.cxf.transport.http.AbstractHTTPDestination.invoke(AbstractHTTPDestination.java:253)
	at org.apache.cxf.transport.servlet.ServletController.invokeDestination(ServletController.java:234)
	at org.apache.cxf.transport.servlet.ServletController.invoke(ServletController.java:208)
	at org.apache.cxf.transport.servlet.ServletController.invoke(ServletController.java:160)
	at org.apache.cxf.transport.servlet.CXFNonSpringServlet.invoke(CXFNonSpringServlet.java:180)
	at org.apache.cxf.transport.servlet.AbstractHTTPServlet.handleRequest(AbstractHTTPServlet.java:298)
	at org.apache.cxf.transport.servlet.AbstractHTTPServlet.doPost(AbstractHTTPServlet.java:217)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:707)
	at org.apache.cxf.transport.servlet.AbstractHTTPServlet.service(AbstractHTTPServlet.java:273)
	at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:812)
	at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:587)
	at org.ops4j.pax.web.service.jetty.internal.HttpServiceServletHandler.doHandle(HttpServiceServletHandler.java:70)
	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:143)
	at org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:577)
	at org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:223)
	at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1127)
	at org.ops4j.pax.web.service.jetty.internal.HttpServiceContext.doHandle(HttpServiceContext.java:271)
	at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:515)
	at org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:185)
	at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1061)
	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:141)
	at org.ops4j.pax.web.service.jetty.internal.JettyServerHandlerCollection.handle(JettyServerHandlerCollection.java:80)
	at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:97)
	at org.eclipse.jetty.server.Server.handle(Server.java:499)
	at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:311)
	at org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:257)
	at org.eclipse.jetty.io.AbstractConnection$2.run(AbstractConnection.java:544)
	at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:635)
	at org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:555)
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.RuntimeException: org.eclipse.egit.github.core.client.RequestException: Validation Failed (422): body is too long (maximum is 65536 characters)
	at org.pentaho.build.buddy.bundles.source.github.GithubSourceRetriever.onStop(GithubSourceRetriever.java:263)
	at org.pentaho.build.buddy.bundles.source.github.SlackbotGithubSourceRetriever.onStop(SlackbotGithubSourceRetriever.java:36)
	at Proxy8365684b_d412_45d9_90d2_6133f10bf378.onStop(Unknown Source)
	at org.pentaho.build.buddy.bundles.orchestrator.OrchestratorImpl.orchestrate(OrchestratorImpl.java:353)
	... 41 more
Caused by: org.eclipse.egit.github.core.client.RequestException: Validation Failed (422): body is too long (maximum is 65536 characters)
	at org.eclipse.egit.github.core.client.GitHubClient.createException(GitHubClient.java:552)
	at org.eclipse.egit.github.core.client.GitHubClient.sendJson(GitHubClient.java:643)
	at org.eclipse.egit.github.core.client.GitHubClient.post(GitHubClient.java:757)
	at org.eclipse.egit.github.core.service.IssueService.createComment(IssueService.java:813)
	at org.eclipse.egit.github.core.service.IssueService.createComment(IssueService.java:785)
	at org.eclipse.egit.github.core.service.IssueService.createComment(IssueService.java:770)
	at org.pentaho.build.buddy.bundles.source.github.GithubSourceRetriever.onStop(GithubSourceRetriever.java:242)
	... 44 more

Copy link
Contributor

@mbatchelor mbatchelor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey Daniel,

Your changes are failing checkstyle. Can you please run the command-line command mvn site which will create the folder target/site. Open the file checkstyle.html in a browser to see how to edit your changes so that they conform to our styles. Thanks!

@wingman-pentaho
Copy link
Collaborator

Hello, This is an automated comment from our build system. Since this pull request originated from a user not in the Pentaho organization, we will not proceed with automatic validation of the pull request. However, rest assured that it will be reviewed manually as soon as we get the chance. Thanks for your contribution!

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy
Copy link
Collaborator

❌ Build failed in 1h 45m 9s

Build command:

mvn clean verify -B -e -Daudit -amd

❗ No tests found!

Errors:

Filtered log (click to expand)

script returned exit code 1

ℹ️ This is an automatic message

buildguy pushed a commit that referenced this pull request Jan 24, 2024
[CLEANUP] Update stage names in the QAT master pipeline
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants