Hao,

Can you clarify the changes you propose for MPath class and related
package.jdo changes for SENTRY-1587?

You suggest changing the identity type from "database" to "application",
but the pathId is not initialized in constructor and not assigned anywhere.
What is your intention here?

Also you changed the equals() method to include ID. What kind of equality
do you require - equality by path value or equality by identity? What is
your thinking here?

This is in regards to the code review https://reviews.apache.org/r/58412.

Thanks,

- Alex

Reply via email to