azotcsit edited a comment on pull request #1208: URL: https://github.com/apache/cassandra/pull/1208#issuecomment-948054593
> Sorry, this is the last round I'm sure... No worries! More rounds we have better the results are :+1: > When actually running C* itself, either from the source tree or the binary distro, Hadoop and its deps aren't on the classpath so we get a NoClassDefFoundError when deleting from the vtables. Oh.. it is a pity that the build does not guarantee correctness of the code.. I think once we migrate from `ant`, it is worth to consider moving Hadoop stuff to a separate module. Anyway, it is too early to think about it. Great catch! Done. > One other related thing I noticed while testing this is that nodetool invalidatepermissionscache <role> has no effect. Oh! I fixed it and added a corresponding test. Thanks for spotting the issue! PS: I triggered a new CI build for the latest changes: https://ci-cassandra.apache.org/view/patches/job/Cassandra-devbranch/1223/ **UPDATE** CI looks good -- 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]
