[GitHub] [drill] vvysotskyi commented on a change in pull request #1712: DRILL-7079: Drill can't query views from the S3 storage when plain authentication is enabled

[GitBox <gi...@apache.org>]

vvysotskyi commented on a change in pull request #1712: DRILL-7079: Drill can't query views from the S3 storage when plain authentication is enabled
URL: https://github.com/apache/drill/pull/1712#discussion_r267691682
 
 

 ##########
 File path: exec/java-exec/src/main/java/org/apache/drill/exec/dotdrill/DotDrillFile.java
 ##########
 @@ -55,6 +56,13 @@ public DotDrillType getType(){
    * @return Return owner of the file in underlying file system.
    */
   public String getOwner() {
+    if (type == DotDrillType.VIEW && status.getOwner().isEmpty()) {
 
 Review comment:
   What about regular tables, parquet metadata cache files or stat files? Is it possible that empty `status.getOwner()` may cause problems for them?

----------------------------------------------------------------
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 go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services