Bowen Liang
Bowen Liang
cc @Yeuoly @takatost
> Please consider using codecov instead of inventing wheel again This PR is not inventing the wheel again, but generally reused the pytest framework and coverage outputs. And it's not...
Why are you so aggressive in your sords to the goal while ignoring the key points I just mentioned about the balance of third-party platform constraint and the feasibility? Is...
PLEASE at least respect the Dify project and the Dify community behaviour code. I raised this PR with proper explanation and implementation for mutual benefit. If you have a better...
I agree with this PR overall, but can you summarize the changes and refactoring design in the description for clarification ? Including but not limited to, the purpose and goals...
+1 for AbstractOperation::close.
> I suggest removing the snappy support, comparing to other formats, I don't see advantage of snappy. > > * gzip: high compression ratio, used widely, many languages and platforms...
Could you append a full screenshot to the desciption for demonstrating the dashboard? And any further detailed explanation for datasource settings, metric group and etc.
> > Currently, the default value of `kyuubi.metrics.reporters` is `JSON`, as PROMETHEUS becomes more popular today, how do you think changing it to `PROMETHEUS` in 1.8? > > @yaooqinn @bowenliang123...
Minor warning message thrown in logs, when compilation on Scala 2.13 : `Error: -release is only supported on Java 9 and higher` (https://github.com/apache/kyuubi/actions/runs/8964924157/job/24623089910#step:8:138) It's repeatedly shown in logs. It's not...