-
Notifications
You must be signed in to change notification settings - Fork 513
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix compiler warnings #4934
Fix compiler warnings #4934
Conversation
9f5d377
to
f3f80a7
Compare
Seems to have broken some tests ;) |
Yes. will unset draft as soon as those are fixed. |
5347c37
to
f764923
Compare
@@ -22,7 +22,7 @@ import com.spotify.scio.coders.Coder | |||
import com.google.common.{hash => g} | |||
import org.apache.beam.sdk.coders.CustomCoder | |||
|
|||
class GuavaBloomFilterCoder[T](implicit val funnel: g.Funnel[T]) | |||
class GuavaBloomFilterCoder[T](implicit val funnel: g.Funnel[_ >: T]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BLoomFilter[String]
can be created from a Funnel[CharSequence]
(expected funnel is <? extends T>
).
Associated coder should also be able to be constructed with such funnel
Codecov Report
@@ Coverage Diff @@
## v0.14.x #4934 +/- ##
===========================================
+ Coverage 62.98% 63.23% +0.25%
===========================================
Files 282 284 +2
Lines 10648 10669 +21
Branches 789 783 -6
===========================================
+ Hits 6707 6747 +40
+ Misses 3941 3922 -19
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
love it
Fix compiler warnings
IsJavaBean
evidence)@uncheked
annotations in pattern matching when required