You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2021/03/09 22:54:23 UTC

[GitHub] [iceberg] RussellSpitzer opened a new issue #2309: Display Table Location in Spark SQL Properties

RussellSpitzer opened a new issue #2309:
URL: https://github.com/apache/iceberg/issues/2309


   I realized today that while you are able to set the location of an Iceberg Table in Spark3, you cannot then view that location using the standard SQL commands.
   
   Maybe we should considering populating this in table properties?


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] aokolnychyi commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
aokolnychyi commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-794730662


   I feel it will be useful.


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] aokolnychyi commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
aokolnychyi commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-835905236


   Closing this one as it has been fixed.


-- 
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] RussellSpitzer commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
RussellSpitzer commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-794588739


   I would also tag this as "good first ticket" but I do not know how to add labels


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] flyrain commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
flyrain commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-797839663


   Looking at this.


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] flyrain commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
flyrain commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-835852899


   Hi @somayaj , the PR #2330 has been merged into the master branch. Please have a try.


-- 
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] aokolnychyi closed issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
aokolnychyi closed issue #2309:
URL: https://github.com/apache/iceberg/issues/2309


   


-- 
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] aokolnychyi commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
aokolnychyi commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-794769981


   I think we can simply adapt `SparkTable$properties`.


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] aokolnychyi commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
aokolnychyi commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-794759953


   @flyrain, would you be interested in working on this?


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] RussellSpitzer commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
RussellSpitzer commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-794770142


   Totally agree, I was worried about it being mis used somewhere, but I think this is the right thing to do


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] RussellSpitzer edited a comment on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
RussellSpitzer edited a comment on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-794770142


   Totally agree, I was worried about it being mis used somewhere, but I think this is the right thing to do, and of course we should add it to "reserved properties" too because currently you can set this but it has no effect


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] aokolnychyi commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
aokolnychyi commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-794768956


   As far as I see, Spark passes the location as a table property. I think we can fake it in `SparkTable` like we fake the provider and the current snapshot id.


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] aokolnychyi commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
aokolnychyi commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-794764189


   We need to check if this will have any possible implications. 


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] somayaj commented on issue #2309: Display Table Location in Spark SQL Properties

Posted by GitBox <gi...@apache.org>.
somayaj commented on issue #2309:
URL: https://github.com/apache/iceberg/issues/2309#issuecomment-835734020


   Can this be worked?


-- 
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org