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 2020/08/18 20:13:02 UTC
[GitHub] [iceberg] jzhuge opened a new pull request #1357: Skip building empty jars
jzhuge opened a new pull request #1357:
URL: https://github.com/apache/iceberg/pull/1357
Skip building the empty jars that we don't need.
Verified the shadow jars still built and no empty jars.
----------------------------------------------------------------
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] openinx commented on pull request #1357: Skip building empty jars
Posted by GitBox <gi...@apache.org>.
openinx commented on pull request #1357:
URL: https://github.com/apache/iceberg/pull/1357#issuecomment-677263670
@jzhuge Did you find that the intellij IDE could not import the relocated guava classes after applied this patch ? Once applied this patch in my own repo, the relocated package (such as `org.apache.iceberg.relocated.com.google.common.base.Preconditions` ) could not be imported, but I could run unit tests in my ide and build the project under my host.
----------------------------------------------------------------
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] jzhuge commented on pull request #1357: Skip building empty jars
Posted by GitBox <gi...@apache.org>.
jzhuge commented on pull request #1357:
URL: https://github.com/apache/iceberg/pull/1357#issuecomment-677430580
Confirmed what you saw. Very strange. Anybody know why?
Your PR LGTM.
On Wed, Aug 19, 2020 at 11:51 PM openinx <no...@github.com> wrote:
> The simple pr (#1360 <https://github.com/apache/iceberg/pull/1360>) fixed
> my package import issue, but i'm not familiar with gradle building, is it
> the right direction to fix this issue ?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/iceberg/pull/1357#issuecomment-677359166>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAOL5JCD4G47Z73NASBYYLDSBTBWZANCNFSM4QEBJXYQ>
> .
>
--
John Zhuge
----------------------------------------------------------------
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] jzhuge commented on pull request #1357: Skip building empty jars
Posted by GitBox <gi...@apache.org>.
jzhuge commented on pull request #1357:
URL: https://github.com/apache/iceberg/pull/1357#issuecomment-677760886
The dependency on bundled-guava is declared using the following syntax
(suggested by Shadow Plugin doc):
compile project(path: ':iceberg-bundled-guava', configuration: 'shadow')
Maybe Idea does not interpret it properly?
On Thu, Aug 20, 2020 at 1:06 AM openinx <no...@github.com> wrote:
> There's a GuavaClasses inside the empty jar. Without that class, then
> bundled-guava won't reference any classes from guava ? finally, the
> relocate won't have effect in this module. But I don't understand why could
> we build this project in terminal.
>
> public class GuavaClasses {
>
> /* * Referencing Guava classes here includes them in the minimized and relocated Guava jar */
> static {
> VisibleForTesting.class.getName();
> Joiner.class.getName();
> MoreObjects.class.getName();
> Objects.class.getName();
> Preconditions.class.getName();
> Splitter.class.getName();
> Throwables.class.getName();
> BiMap.class.getName();
> FluentIterable.class.getName();
> ImmutableBiMap.class.getName();
> ImmutableList.class.getName();
> ImmutableMap.class.getName();
> ImmutableSet.class.getName();
> Iterables.class.getName();
> Iterators.class.getName();
> ListMultimap.class.getName();
> Lists.class.getName();
> MapMaker.class.getName();
> Maps.class.getName();
> Multimap.class.getName();
> Multimaps.class.getName();
> Ordering.class.getName();
> Sets.class.getName();
> Streams.class.getName();
> Hasher.class.getName();
> HashFunction.class.getName();
> Hashing.class.getName();
> Files.class.getName();
> Bytes.class.getName();
> MoreExecutors.class.getName();
> ThreadFactoryBuilder.class.getName();
> Iterables.class.getName();
> }
>
> }
>
> [image: image]
> <https://user-images.githubusercontent.com/5028729/90732685-0adc4600-e2fe-11ea-9009-875ef9a40a8d.png>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/iceberg/pull/1357#issuecomment-677445102>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAOL5JFXRDLHZQ2QNMN2JULSBTKRNANCNFSM4QEBJXYQ>
> .
>
--
John Zhuge
----------------------------------------------------------------
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] rdblue commented on pull request #1357: Skip building empty jars
Posted by GitBox <gi...@apache.org>.
rdblue commented on pull request #1357:
URL: https://github.com/apache/iceberg/pull/1357#issuecomment-675740265
Looks great! Thanks @jzhuge!
----------------------------------------------------------------
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] rdblue commented on pull request #1357: Skip building empty jars
Posted by GitBox <gi...@apache.org>.
rdblue commented on pull request #1357:
URL: https://github.com/apache/iceberg/pull/1357#issuecomment-677764793
Looks like IntelliJ relies on the 'empty' classifier somehow. Probably not worth the trouble to find out why when we can just add the classifier back. I've updated the PR to use `archiveClassifier` and will merge when it passes tests.
----------------------------------------------------------------
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] openinx commented on pull request #1357: Skip building empty jars
Posted by GitBox <gi...@apache.org>.
openinx commented on pull request #1357:
URL: https://github.com/apache/iceberg/pull/1357#issuecomment-677359166
The simple pr (https://github.com/apache/iceberg/pull/1360) fixed my package import issue, but i'm not familiar with gradle building, is it the right direction to fix this issue ?
----------------------------------------------------------------
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] rdblue merged pull request #1357: Skip building empty jars
Posted by GitBox <gi...@apache.org>.
rdblue merged pull request #1357:
URL: https://github.com/apache/iceberg/pull/1357
----------------------------------------------------------------
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] openinx commented on pull request #1357: Skip building empty jars
Posted by GitBox <gi...@apache.org>.
openinx commented on pull request #1357:
URL: https://github.com/apache/iceberg/pull/1357#issuecomment-677445102
There's a `GuavaClasses` inside the empty jar. Without that class, then `bundled-guava` won't reference any classes from guava ? finally, the relocate won't have effect in this module. But I don't understand why could we build this project in terminal.
```java
public class GuavaClasses {
/*
* Referencing Guava classes here includes them in the minimized and relocated Guava jar
*/
static {
VisibleForTesting.class.getName();
Joiner.class.getName();
MoreObjects.class.getName();
Objects.class.getName();
Preconditions.class.getName();
Splitter.class.getName();
Throwables.class.getName();
BiMap.class.getName();
FluentIterable.class.getName();
ImmutableBiMap.class.getName();
ImmutableList.class.getName();
ImmutableMap.class.getName();
ImmutableSet.class.getName();
Iterables.class.getName();
Iterators.class.getName();
ListMultimap.class.getName();
Lists.class.getName();
MapMaker.class.getName();
Maps.class.getName();
Multimap.class.getName();
Multimaps.class.getName();
Ordering.class.getName();
Sets.class.getName();
Streams.class.getName();
Hasher.class.getName();
HashFunction.class.getName();
Hashing.class.getName();
Files.class.getName();
Bytes.class.getName();
MoreExecutors.class.getName();
ThreadFactoryBuilder.class.getName();
Iterables.class.getName();
}
}
```
![image](https://user-images.githubusercontent.com/5028729/90732685-0adc4600-e2fe-11ea-9009-875ef9a40a8d.png)
----------------------------------------------------------------
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