SandishKumarHN commented on issue #8558: 7227 : Prohibit Non Final Static Field
URL: https://github.com/apache/incubator-druid/pull/8558#issuecomment-542450809
@leventov inspection failure from
`
SeekableStreamSupervisor.java (4)
175: TaskGroup() Optional used as type for
SandishKumarHN commented on issue #8558: 7227 : Prohibit Non Final Static Field
URL: https://github.com/apache/incubator-druid/pull/8558#issuecomment-537292221
@leventov looks like a code inspection issue, which is not related to this
PR. verified the code I don't see any duplicate code.
SandishKumarHN commented on issue #8558: 7227 : Prohibit Non Final Static Field
URL: https://github.com/apache/incubator-druid/pull/8558#issuecomment-535793321
@leventov fixed
This is an automated message from the Apache Git
SandishKumarHN commented on issue #8558: 7227 : Prohibit Non Final Static Field
URL: https://github.com/apache/incubator-druid/pull/8558#issuecomment-535237459
@leventov "Inspections: pull requests (Druid) — TeamCity build failed" is
not related to PR.
SandishKumarHN commented on issue #8558: 7227 : Prohibit Non Final Static Field
URL: https://github.com/apache/incubator-druid/pull/8558#issuecomment-534863961
@leventov made changes
This is an automated message from the
SandishKumarHN commented on issue #8558: 7227 : Prohibit Non Final Static Field
URL: https://github.com/apache/incubator-druid/pull/8558#issuecomment-533846781
@leventov thanks for the review, made changes based your suggestions.
SandishKumarHN commented on issue #8558: 7227 : Prohibit Non Final Static Field
URL: https://github.com/apache/incubator-druid/pull/8558#issuecomment-533739524
@leventov can you please review this also.
This is an automated
SandishKumarHN commented on issue #8558: 7227 : Prohibit Non Final Static Field
URL: https://github.com/apache/incubator-druid/pull/8558#issuecomment-533247091
@ccaominh Thanks for quick comments changed, please review now