Zixuan Liu comments

Results 213 comments of


                                            Zixuan Liu

fix: add id field to bookie socket address struct

@zymap I can add a test to check the id field or host and port field exists.

fix: add id field to bookie socket address struct

@zymap Please take a look.

[Issue 827][bugfix] fix error delay of default backoff policy

Closed by #834.

`--output` doesn't work in `pulsarctl token` command

I submitted #https://github.com/streamnative/pulsarctl/pull/608 to fix this, could you review this?

[Issue 489] TLS fix for certificate that does not include IP SANS, save hostname before switching to physical addr

Fix https://github.com/apache/pulsar-client-go/issues/489. This is a good idea, you can also set `AllowInsecureConnection: true`.

[Issue 489] TLS fix for certificate that does not include IP SANS, save hostname before switching to physical addr

Ping @merlimat

Connect to multi hosts

Do you want to manage multiple hosts? pulsarctl supports configuring multiple contexts, please checkout the help message by `pulsarctl context -h`.

An update to a sink shouldn't have a default value in AutoAck or Retain Ordering

@AlvaroStream Thanks for your report! Please keep the same value when updating to avoid this issue. Because of to limit by the cli, we cannot provide a null value.

bk ledger get <ledgerId> output ensemble with empty hostname and port

@Shawyeok Thanks for your report! I made a PR for this: https://github.com/streamnative/pulsarctl/pull/545, please take a look if you are free.

Enable the race detector in the test environment

@timmyyuan Please fix the CI. go test: -race requires cgo; enable cgo by setting CGO_ENABLED=1