[GitHub] [beam] Lizzfox commented on pull request #22399: [CdapIO] Fixed necessary warnings

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

Lizzfox commented on PR #22399:
URL: https://github.com/apache/beam/pull/22399#issuecomment-1192529993

   > There are also other suppression annotations exists currently, are there also considered need to be resolved?
   
   @Abacn, thank you for the comment. Yes, there are two classes with warnings suppression left in the CdapIO package. 
   1. In the `Plugin` class, the warnings will be fixed in the following PR where we will integrate SparkReceiverIO with CdapIO.
   
   2. The `BatchContextImpl` class is an abstraction that we've implemented to make it possible to use CDAP abstractions in the Apache Beam context. Methods that produce warnings are overridden so we can't change their signature to resolve warnings. The suppressions in the `BatchContextImp` class aren't expected to be resolved anytime soon.


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

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org