Ivan Zhakov wrote:
>>>> URL: http://svn.apache.org/r1649029
>>>> Log:
>>>> Sync FSX with FSFS: Merge DAG-related sub-pool introduction
>>>> patches r1647905, r1648243, r1648253 and r1648272 from FSFS
>>>> and resolve the usual text conflicts due to naming differences.
>>>> Block revisions r1648230, r1648238, r1648241, r1648242 and r1648532.

> Also, it seems that this commit has another problems:
> 1. It doesn't have proper log message [1].
> 2. It mixes different unrelated changes in one commit, while it will
> be much easier to review them separately.

There is nothing wrong here. The log message says what's merged, and that is 
four revisions all described as "DAG-related sub-pool introduction", which 
means they are all parts of the same change - not unrelated changes.

- Julian

Reply via email to