mck

Results 107 comments of mck

Yes, please squash to one commit. Are you familiar with our commit message format? (if not, the committer will adjust it accordingly) There's also one minor edit to make, the...

ci-cassandra: https://ci-cassandra.apache.org/job/Cassandra-devbranch-artifacts/1749/ LGTM šŸ‘

latest changed LGTM! thanks! https://ci-cassandra.apache.org/job/Cassandra-devbranch/1886/

> https://ci-cassandra.apache.org/job/Cassandra-devbranch/1886/ something is up with the dtest-upgrade… (took a re-run of it here too: https://ci-cassandra.apache.org/job/Cassandra-devbranch-dtest-upgrade/1177/ ) haven't had time to investigate it… but wondering if this patch needs to...

new round: https://ci-cassandra.apache.org/blue/organizations/jenkins/Cassandra-devbranch/detail/Cassandra-devbranch/1900/pipeline

This replaces our existing dependency management, found in one place [here](https://github.com/apache/cassandra/blob/trunk/build.xml#L504-L874), with three explicit pom files. This makes dependency management harder for the committers and contributors, and it will introduce...

> The plexus-utils2 dependency is a transitive dependency included in the last released version of Maven Ant Tasks Yes, plexus is inside the jar file. > I considered authoring a...

> Maven pom files are well known and well understood, so easier to reason about than our current model. Ok, thanks for chiming in @dcapwell :) Before we commit to...

> Do you have any criteria for comparing formats, besides personal preference? Sure, we need to stay away from "it's my favourite build tool" sentiments, and focus on what is...

> merging into 1 pom could be a problem as we publish more than one: https://repo1.maven.org/maven2/org/apache/cassandra/, we publish cassandra-all and cassandra-parent currently... there's also the `tmp-apache-cassandra-*-deps.pom` which we generate and...