lavanyavijayk commented on PR #2722: URL: https://github.com/apache/incubator-hugegraph/pull/2722#issuecomment-2599647375
> @lavanyavijayk There is an issue with the version number [here](https://github.com/apache/incubator-hugegraph/blob/master/hugegraph-pd/hg-pd-service/pom.xml#L127) for compilation error, it should be updated to `${hugegraph-commons.version}`. Thank you for pointing that out. I have made the changes. I also want to get more details for the following. The ticket mentions a change in /hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/version/ApiVersion.java, but it is unclear which string should be replaced with the revision value. Could you confirm this before proceeding? I attempted to dynamically assign the revision value to CoreVersion.DEFAULT_VERSION (/hugegraph-server/hugegraph-core/src/main/java/org/apache/hugegraph/version/CoreVersion.java), but it caused a compilation error. The issue arises because CoreVersion.DEFAULT_VERSION is also used as an annotation value in /hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/server/ApplicationConfig.java, which requires a constant expression. Should I defer this change for now? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
