Doroszlai, Attila comments

Results 127 comments of


                                            Doroszlai, Attila

[Recon] Dependabot Package Upgrade: Bump the minor-updates group in /hadoop-ozone/recon/src/main/resources/webapps/recon/ozone-recon-web with 17 updates

@devabhishekpal

[Recon] Dependabot Package Upgrade: Bump the minor-updates group in /hadoop-ozone/recon/src/main/resources/webapps/recon/ozone-recon-web with 17 updates

new PR: #6365

[fix] failing Yarn test, false negative check result and artifact upload

@elek please review

HDDS-7329. Extend ozone admin datanode usageinfo and list info to accept hostname parameter

@xBis7 I plan to finish this, are you OK with that?

HDDS-7329. Extend ozone admin datanode usageinfo and list info to accept hostname parameter

> As far as I can recall, these were the unresolved issues that blocked this ticket Thanks @xBis7, that's very useful for the future. However, here I don't intend to...

HDDS-7329. Extend ozone admin datanode usageinfo and list info to accept hostname parameter

@xBis7 please let me know if you are OK with the latest update to be merged

HDDS-7329. Extend ozone admin datanode usageinfo and list info to accept hostname parameter

Thanks @xBis7 for the patch, @kerneltime, @sadanand48, @sokui, @Xushaohong for the review.

HDDS-7228. ChecksumByteBufferImpl.update() is expensive

> I suspect it is the warning message that is breaking the acceptance tests, as they are likely expecting a certain output on stdout / stderr and this warning is...

HDDS-7228. ChecksumByteBufferImpl.update() is expensive

@jojochuang I've added the JVM arg to avoid the warning, plus changed initialization of the `Field` to `static`, please take a look if you're OK with that. I think we...

HDDS-7228. ChecksumByteBufferImpl.update() is expensive

> We should probably move these to a log message, rather than printing directly to stdout / stderr, which is what I assume e.printStackTrace() does. Done.