Yida Wu has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17572 )

Change subject: IMPALA-10636: Add testcases for abnormal cases of spilling to a 
remote filesystem
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/17572/1/be/src/runtime/io/disk-file.h
File be/src/runtime/io/disk-file.h:

http://gerrit.cloudera.org:8080/#/c/17572/1/be/src/runtime/io/disk-file.h@154
PS1, Line 154:
> Is there no other way to specify an invalid path? Is the issue that we will
Done, construct new TmpFile objects with an invalid path to create the error.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I600f93d67ba2267c14d179822056a24bedca31ef
Gerrit-Change-Number: 17572
Gerrit-PatchSet: 2
Gerrit-Owner: Yida Wu <wydbaggio...@gmail.com>
Gerrit-Reviewer: Abhishek Rawat <ara...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Yida Wu <wydbaggio...@gmail.com>
Gerrit-Comment-Date: Mon, 14 Jun 2021 17:59:59 +0000
Gerrit-HasComments: Yes

Reply via email to