michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-807930192
/pulsarbot run-failure-checks
--
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 g
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-806392323
/pulsarbot run-failure-checks
--
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 g
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-804614255
/pulsarbot run-failure-checks
--
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 g
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-801621751
Had merge conflicts. Fixed with a rebase.
This is an automated message from the Apache Git Service.
To r
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-800781010
/pulsarbot run-failure-checks
This is an automated message from the Apache Git Service.
To respond to th
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-799982394
Failed checkstyle. I should be able to fix it tomorrow morning.
This is an automated message from the Ap
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-799972743
In putting together the test, I noticed that the implementation would be
cleaner with new class and test just that class, so I refactored the original
change.
@si
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-797850790
/pulsarbot run-failure-checks
This is an automated message from the Apache Git Service.
To respond to th
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-797261044
> Maybe we could find a way to make the same offloaders nar packages in the
different directories only be loaded once. For example, in the PulsarService
one Offloaders f
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-796868189
> Users could change the offload directory in the namespace level or the
topic-level through the admin API, right?
Yes, the the `offloadersDirectory` can be overri
michaeljmarshall commented on pull request #9878:
URL: https://github.com/apache/pulsar/pull/9878#issuecomment-796442155
Failed the check style. I'll be able to take a look and fix it in an hour.
This is an automated message
11 matches
Mail list logo