waukin commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2562758155
> You can leave the existing method as `output(catalogs)` and just have that
method output "No catalogs exist." if null is passed into it.
I’ve kept the existing method as `ou
justinmclean commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2562015624
You can leave the existing method as `output(catalogs)` and just have that
method output "No catalogs exist." if null is passed into it.
--
This is an automated message from
waukin commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2561575925
Hi @Abyss-lord, I see. We can handle that by having the `output` method
accept a string argument and simply printing it. I'll open another issue to
make it clearer.
--
This is an
Abyss-lord commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2561525035
Hi @waukin , In the `ListXXX` method, if there is No data, CLI should give a
hint, such as "No metalakes exist. "The current implementation is implemented
by two methods: `Syste
waukin commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2561154657
Hi @Abyss-lord, I’m still confused about the requirement that the `output`
method should accept string argument. In the previous implementation, the
`output` method accepted differe
Abyss-lord commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2560574358
> > @waukin BTW, `output` method should accept string argument, Please amend
it here.
waukin commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2560513458
@xunliu please assign this issue to me.
--
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
waukin commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2560513158
> > Hi @Abyss-lord, I’m halfway through implementing the feature. If you
don’t mind, I’d like to continue working on it.
>
> that's ok, since you wrote the previous code, it w
Abyss-lord commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2560501712
> Hi @Abyss-lord, I’m halfway through implementing the feature. If you don’t
mind, I’d like to continue working on it.
that's ok, since you wrote the previous code, it wou
waukin commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2560467034
Hi @Abyss-lord, I’m halfway through implementing the issue. If you don’t
mind, I’d like to continue working on it.
--
This is an automated message from the Apache Git Service.
To
Abyss-lord commented on issue #5957:
URL: https://github.com/apache/gravitino/issues/5957#issuecomment-2560063480
I would like to work on it.
--
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 s
waukin opened a new issue, #5957:
URL: https://github.com/apache/gravitino/issues/5957
### Describe the feature
The current `SchemaDetails` command in the Gravitino CLI supports only plain
format output.
I plan to add support for table format output to the `SchemaDetails` command.
14 matches
Mail list logo
