nizhikov commented on a change in pull request #8867:
URL: https://github.com/apache/ignite/pull/8867#discussion_r596373385
##########
File path: modules/platforms/dotnet/Apache.Ignite.Core.Tests/Services/Model.cs
##########
@@ -123,7 +123,7 @@ public override int GetHashCode()
// ReSharper disable once InconsistentNaming
public enum ACL
{
- Allow, Deny
+ ALLOW, DENY
Review comment:
Actually, it's kind of one more minor .Net -> Java integration issue.
I'm not aware of naming contract in enum that shared between .Net and Java.
Do you know one?
With previous enum declaration we have an exception in
`ServicesTypeAutoResolveTest#TestCallPlatformServiceRemote`
```
Apache.Ignite.Core.Services.ServiceInvocationException : Proxy method
invocation failed with an exception. Examine InnerException for details.
----> Apache.Ignite.Core.Common.IgniteException : Conflicting enum values.
Name 'Allow' uses ordinal value (0) that is also used for name 'ALLOW'
[typeName='org.apache.ignite.platform.model.ACL']
----> Apache.Ignite.Core.Common.JavaException : class
org.apache.ignite.IgniteException: Conflicting enum values. Name 'Allow' uses
ordinal value (0) that is also used for name 'ALLOW'
[typeName='org.apache.ignite.platform.model.ACL']
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
