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

Add Geode Provider #235

Closed
wants to merge 10 commits into from
Closed

Add Geode Provider #235

wants to merge 10 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Apr 20, 2020

No description provided.

@ghost ghost mentioned this pull request Apr 20, 2020
@lukas-krecan
Copy link
Owner

Thanks a lot, I will look at it in next few days.

@lukas-krecan
Copy link
Owner

I am not able to run the test. Locally it throws java.lang.NoClassDefFoundError: org/apache/geode/management/internal/cli/result/ResultData and on CI it fails like this https://travis-ci.org/github/lukas-krecan/ShedLock/jobs/677620230 Can you please fix it?

@codecov-io
Copy link

codecov-io commented Apr 22, 2020

Codecov Report

Merging #235 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #235   +/-   ##
=========================================
  Coverage     86.02%   86.02%           
  Complexity      334      334           
=========================================
  Files            67       67           
  Lines          1331     1331           
  Branches         69       69           
=========================================
  Hits           1145     1145           
  Misses          163      163           
  Partials         23       23           
Impacted Files Coverage Δ Complexity Δ
...mbs/shedlock/provider/mongo/MongoLockProvider.java 86.66% <0.00%> (ø) 8.00% <0.00%> (ø%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a3f6d0...6e6b1d8. Read the comment docs.

@ghost
Copy link
Author

ghost commented Apr 22, 2020

I think the build failed as the module wasnt mentioned in pom.xml , wondering how it got picked up for build and test , anyways have updated the blocking unlock to asynchronous unlock.
Have also updated the pom.xml.

Please take a look .

@ghost
Copy link
Author

ghost commented Apr 22, 2020

Ok sorted out the java docs error , checking on the jacoco coverage error ( plugin is saying unable to create coverage , but works fine on local ) , probably as the test starts vm outside of the Testcase , probably its not able to pick up the coverage.

Also i see we have used jupiter for test rules , somehow intellij doesnt like it and complains for each testSuite that the jupiter-platform jar is unavailable , any suggestions ?

@lukas-krecan
Copy link
Owner

Thank, I will look at it over weekend.

@lukas-krecan
Copy link
Owner

Hi, it seems that the tests still do not work. If I execute GeodeLockProviderIntegrationTest it does not run any tests. Can you please fix it? The cause might be that the test is mixing JUnit 4 and 5 annotations.

@ghost
Copy link
Author

ghost commented Apr 25, 2020

Looking at it now .

@ghost
Copy link
Author

ghost commented Apr 25, 2020

Hi Lukas,
The build is failing on travis with the error "maximum" log limit exceed and terminating the jobs.Please can you increase the log limit ? If not possible to increase length , can comment out all other modules

@lukas-krecan
Copy link
Owner

Thanks a lot, will take a look at it

@lukas-krecan
Copy link
Owner

Some of the builds are passing, so feel free to ignore the issue. I will fix it later.

@ghost
Copy link
Author

ghost commented Apr 25, 2020

Ok thanks

Reference tried to fix openJDK issue , https://travis-ci.org/github/lukas-krecan/ShedLock/builds/679451966

However i had to downgrade geode to 1.9.2 , so if we can solve the openJDK issue then please let me know , ideally i would want to use 1.10 as currently 1.12 is latest.

@lukas-krecan
Copy link
Owner

Hi, I am really sorry but Geode is so complex that I am afraid I would not be able to maintain it. I would recommend to create a third party lock provider similar to the one for CosmosDB https://github.com/jesty/shedlock-provider-cosmosdb

@ghost
Copy link
Author

ghost commented May 3, 2020

Hi
Not sure what the issue , jdk8 build is passing and the underlying complexity is same as hazelcast.
If you want I can take a look at openJdK build again.

@lukas-krecan
Copy link
Owner

I managed to get the tests running with test containers but the necessity to deploy the DistributedLockFunction makes automated testing and deployment quite complex. There is nothing like that needed in Hazelcast. I am afraid I will not be able to maintain this code so it would be better if you published it as third party module.

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.

2 participants