HI,

Thanks for the review.

> 1. There're empty "build" directories which are introduced by gradle build,
> these should be removed as a source package.

These have been removed.

> 2. The python package version is 0.9.0, which is wrong; please check
> "setup.py".

It was correct for the built artifacts, just no in the source, as my PR to fix 
the version numbers has not been merged.

> 3. The version number in the docs is "0.8.1", not "0.8.1-incubating", which
> will be a link issue when the doc is online.

This has also been fixed.

> 4. Also, I just checked the RC2 tag and this commit (
> https://github.com/apache/gravitino/commit/fa8e968f8b78c86cbc91e4badd95379efba2f179),
> it changes the jacoco version, which is not correct, should be reverted.

The Jocco version has been corrected (0.8.10).

Kind Regards,
Justin

Reply via email to