shreya024 commented on pull request #16011: URL: https://github.com/apache/shardingsphere/pull/16011#issuecomment-1066020047
> > Hi @shreya024 , thanks for your contribution, but I found there's some CI didn't pass. e.g. In https://github.com/apache/shardingsphere/runs/5512740096?check_suite_focus=true > > ``` > > Error: assertReleaseLock(org.apache.shardingsphere.data.pipeline.core.lock.PipelineSimpleLockTest) Time elapsed: 0.031 s <<< ERROR! > > java.lang.RuntimeException: Could not get metadata persist service > > at org.apache.shardingsphere.data.pipeline.core.lock.PipelineSimpleLockTest.setUp(PipelineSimpleLockTest.java:39) > > ``` > > > > > > > > > > > > > > > > > > > > > > > > From `PipelineSimpleLock` source code: > > ``` > > private PipelineSimpleLock() { > > Optional<MetaDataPersistService> persistServiceOptional = PipelineContext.getContextManager().getMetaDataContexts().getMetaDataPersistService(); > > persistServiceOptional.orElseThrow(() -> new RuntimeException("Could not get metadata persist service")); > > ... > > ``` > > > > > > > > > > > > > > > > > > > > > > > > We need to prepare `MetaDataPersistService` before `PipelineSimpleLock.getInstance()` invocation. You could reference `LockRegistryServiceTest` and `PipelineContextUtil`. > > And you could run unit test for `shardingsphere-pipeline-test` module before push new commit, it's more effective to get unit test result than running on GitHub CI. > > Feel free to leave comments on this PR if you have any question. > > Okay I'm working on adding MetaDataPersistService. > > Could you explain how to run unit test for a particular module like shardingsphere-pipeline-test on IntelliJ? Could you kindly provide an example on how to exactly prepare metaDataPersistService? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
