Quanlong Huang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18792 )

Change subject: IMPALA-11457 Fix regression with unknown disk id
......................................................................


Patch Set 1:

> Patch Set 1:
>
> > Patch Set 1:
> >
> > Nice finding! Can we reveal the bug using existing test data? Or does it 
> > only happen on Ozone?
>
> I don't know of a case with HDFS where the storage ID is unknown. Maybe?

I think cloud storages like S3 will have unknown disk id as -1. But now sure if 
we actually use it.

If this can't be revealed by existing test data, maybe it won't worth a 
separated patch and we can merge it with the ozone patch.


--
To view, visit http://gerrit.cloudera.org:8080/18792
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I571ac0669ceb6a42561594c3f96723d5ed293902
Gerrit-Change-Number: 18792
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Comment-Date: Thu, 28 Jul 2022 04:02:29 +0000
Gerrit-HasComments: No

Reply via email to