zjffdu commented on PR #4506:
URL: https://github.com/apache/zeppelin/pull/4506#issuecomment-1432962218
Thanks @canhtran for the feedback,
https://github.com/apache/zeppelin/pull/4558 indeed cause this issue. The
workaround is using `mvn clean package -DskipTests -Pflink-113`
--
This is
zjffdu commented on PR #4506:
URL: https://github.com/apache/zeppelin/pull/4506#issuecomment-1427386135
@canhtran How did you build Zeppelin?
--
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
zjffdu commented on PR #4506:
URL: https://github.com/apache/zeppelin/pull/4506#issuecomment-1402059634
@halfvector @zenozeng @PrabhuJoseph @vineethNaroju I have updated this PR
and CI is passed now. Feel free to try it to see whether it solves your
problems.
--
This is an automated
zjffdu commented on PR #4506:
URL: https://github.com/apache/zeppelin/pull/4506#issuecomment-1357122735
Thanks for the feedback @PrabhuJoseph @zenozeng @vineethNaroju , I am busy
on other stuff recently, will take a look at this later.
--
This is an automated message from the Apache Git
zjffdu commented on PR #4506:
URL: https://github.com/apache/zeppelin/pull/4506#issuecomment-1313401152
> BTW, a test related to this change failed. Is it fine?
https://github.com/apache/zeppelin/actions/runs/3448729205/jobs/5755992907
I will fix it
--
This is an automated message
zjffdu commented on PR #4506:
URL: https://github.com/apache/zeppelin/pull/4506#issuecomment-1311634940
> > Will merge if no more comments
>
> No automated tests?
Oops, missing in the commit, let me add it.
--
This is an automated message from the Apache Git Service.
To
zjffdu commented on PR #4506:
URL: https://github.com/apache/zeppelin/pull/4506#issuecomment-1311620071
Will merge if no more comments
--
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
