> On June 8, 2020, 2:39 p.m., Madhan Neethiraj wrote: > > repository/src/main/java/org/apache/atlas/discovery/EntitySearchProcessor.java > > Line 265 (original) > > <https://reviews.apache.org/r/72566/diff/1/?file=2234073#file2234073line265> > > > > Why is this code block removed? If necessary, please rebase.
This code block was showing Dead Code by findbugs, initially line #276 was showing dead code so i removed and then the code above started showing dead code. That is why i had to remove the whole code block. > On June 8, 2020, 2:39 p.m., Madhan Neethiraj wrote: > > repository/src/main/java/org/apache/atlas/glossary/GlossaryService.java > > Line 905 (original), 905 (patched) > > <https://reviews.apache.org/r/72566/diff/1/?file=2234076#file2234076line905> > > > > This update doesn't look right. Why is 'ret' initialized in #905, and > > have it reassinged in #915. What this the issue flagged for the existing > > code? This code block was initially showing "Useless object stored in variable ret of method" I tried to make less changes to the existing code as possible though i have made better changes now, please have a look. > On June 8, 2020, 2:39 p.m., Madhan Neethiraj wrote: > > repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLLexer.java > > Line 86 (original), 88 (patched) > > <https://reviews.apache.org/r/72566/diff/1/?file=2234081#file2234081line88> > > > > Use of Arrays.copyOf() could cause excessive memory allocation. Why is > > this necessary - in AtlasDSLLexer.java, AtlasDSLParser.java? By statement "return tokenNames;" findBugs was throwing "org.apache.atlas.query.antlr4.AtlasDSLLexer.getTokenNames() may expose internal representation by returning AtlasDSLLexer.tokenNames [org.apache.atlas.query.antlr4.AtlasDSLLexer] At AtlasDSLLexer.java:[line 88] EI_EXPOSE_REP" That is why i had to opt for this solution. - mayank ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72566/#review220966 ----------------------------------------------------------- On June 9, 2020, 12:44 p.m., mayank jain wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72566/ > ----------------------------------------------------------- > > (Updated June 9, 2020, 12:44 p.m.) > > > Review request for atlas, Jayendra Parab, Madhan Neethiraj, Nixon Rodrigues, > and Sarath Subramanian. > > > Bugs: ATLAS-1798 > https://issues.apache.org/jira/browse/ATLAS-1798 > > > Repository: atlas > > > Description > ------- > > Currently Findbugs complaints about some problems (see attachment) in the > repository module. They should be fixed to get the code more reliable. > > > Diffs > ----- > > repository/src/main/java/org/apache/atlas/GraphTransactionInterceptor.java > 57e454a > > repository/src/main/java/org/apache/atlas/discovery/EntityDiscoveryService.java > dd4d1b4 > > repository/src/main/java/org/apache/atlas/discovery/EntitySearchProcessor.java > 56956e6 > > repository/src/main/java/org/apache/atlas/discovery/SearchAggregatorImpl.java > e8f7dbc > repository/src/main/java/org/apache/atlas/discovery/SearchProcessor.java > 804c694 > repository/src/main/java/org/apache/atlas/glossary/GlossaryService.java > d630f66 > repository/src/main/java/org/apache/atlas/glossary/GlossaryTermUtils.java > 2c84ec7 > repository/src/main/java/org/apache/atlas/glossary/GlossaryUtils.java > 2a2cebb > repository/src/main/java/org/apache/atlas/query/AtlasDSL.java b8a744b > repository/src/main/java/org/apache/atlas/query/GremlinQueryComposer.java > 801e898 > repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLLexer.java > 142b9ca > repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParser.java > 6b33edb > > repository/src/main/java/org/apache/atlas/repository/audit/AbstractStorageBasedAuditRepository.java > 1aac375 > > repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditListener.java > 69d373d > > repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditListenerV2.java > 79527ac > > repository/src/main/java/org/apache/atlas/repository/audit/HBaseBasedAuditRepository.java > 9fca744 > > repository/src/main/java/org/apache/atlas/repository/converters/AtlasArrayFormatConverter.java > c335f0a > > repository/src/main/java/org/apache/atlas/repository/converters/AtlasInstanceConverter.java > 6fc0c65 > > repository/src/main/java/org/apache/atlas/repository/converters/AtlasMapFormatConverter.java > 0eacd8e > > repository/src/main/java/org/apache/atlas/repository/converters/AtlasStructFormatConverter.java > ae92b8b > > repository/src/main/java/org/apache/atlas/repository/graph/FullTextMapperV2.java > 497a877 > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java > 4a09b08 > repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java > 7b7ec65 > > repository/src/main/java/org/apache/atlas/repository/impexp/AtlasServerService.java > 542106f > > repository/src/main/java/org/apache/atlas/repository/impexp/ExportService.java > 0491a85 > > repository/src/main/java/org/apache/atlas/repository/impexp/ImportService.java > 1d29bf8 > > repository/src/main/java/org/apache/atlas/repository/impexp/ImportTransforms.java > a2f592c > repository/src/main/java/org/apache/atlas/repository/impexp/ZipSink.java > 6375454 > repository/src/main/java/org/apache/atlas/repository/impexp/ZipSource.java > 812add9 > > repository/src/main/java/org/apache/atlas/repository/impexp/ZipSourceDirect.java > 04342fa > > repository/src/main/java/org/apache/atlas/repository/impexp/ZipSourceWithBackingDirectory.java > 7963800 > > repository/src/main/java/org/apache/atlas/repository/migration/DataMigrationService.java > 0a2257e > > repository/src/main/java/org/apache/atlas/repository/migration/ZipFileMigrationImporter.java > d56261f > > repository/src/main/java/org/apache/atlas/repository/ogm/AtlasServerDTO.java > 2f7ca11 > > repository/src/main/java/org/apache/atlas/repository/store/bootstrap/AtlasTypeDefStoreInitializer.java > 8e7c1b3 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java > 3f8503a > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasClassificationDefStoreV2.java > 9ffede4 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityChangeNotifier.java > 0dc3193 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityDefStoreV2.java > e5153de > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityStoreV2.java > 89076c1 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasRelationshipStoreV2.java > 8d74489 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasStructDefStoreV2.java > 9a45f00 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasTypeDefGraphStoreV2.java > ed17b92 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/BulkImporterImpl.java > 8e17fd4 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphMapper.java > 2ed524f > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphRetriever.java > 757fcb1 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/UniqAttrBasedEntityResolver.java > d1c3bde > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/bulkimport/pc/EntityConsumer.java > b73988f > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/bulkimport/pc/EntityCreationManager.java > 734add6 > repository/src/main/java/org/apache/atlas/util/AtlasMetricsCounter.java > 10319d0 > repository/src/main/java/org/apache/atlas/util/AtlasMetricsUtil.java > beb90e6 > repository/src/main/java/org/apache/atlas/util/FileUtils.java 66ade26 > > repository/src/test/java/org/apache/atlas/discovery/BasicSearchClassificationTest.java > 9b16e91 > repository/src/test/java/org/apache/atlas/query/DSLQueriesTest.java 5ace379 > > repository/src/test/java/org/apache/atlas/query/GremlinQueryComposerTest.java > 959aa11 > repository/src/test/java/org/apache/atlas/repository/AtlasTestBase.java > c2668b1 > > repository/src/test/java/org/apache/atlas/repository/audit/AtlasAuditServiceTest.java > 54f75cc > > repository/src/test/java/org/apache/atlas/repository/audit/AuditRepositoryTestBase.java > bf4f395 > > repository/src/test/java/org/apache/atlas/repository/audit/CassandraAuditRepositoryTest.java > 26d3a60 > > repository/src/test/java/org/apache/atlas/repository/audit/InMemoryAuditRepositoryTest.java > 3bdfcf9 > > repository/src/test/java/org/apache/atlas/repository/impexp/AtlasServerServiceTest.java > 91ffc27 > > repository/src/test/java/org/apache/atlas/repository/impexp/ExportIncrementalTest.java > 0e3955d > > repository/src/test/java/org/apache/atlas/repository/impexp/ExportServiceTest.java > 8e19dc4 > > repository/src/test/java/org/apache/atlas/repository/impexp/ImportTransformsShaperTest.java > c2b9dbb > > repository/src/test/java/org/apache/atlas/repository/impexp/ImportTransformsTest.java > 1959576 > > repository/src/test/java/org/apache/atlas/repository/impexp/IncrementalExportEntityProviderTest.java > ed6c12d > > repository/src/test/java/org/apache/atlas/repository/impexp/RelationshipAttributesExtractorTest.java > 5f41cc9 > > repository/src/test/java/org/apache/atlas/repository/impexp/ReplicationEntityAttributeTest.java > ebdc7b5 > > repository/src/test/java/org/apache/atlas/repository/impexp/UniqueListTest.java > 2118df9 > > repository/src/test/java/org/apache/atlas/repository/impexp/ZipDirectTest.java > faa31c3 > > repository/src/test/java/org/apache/atlas/repository/impexp/ZipSinkTest.java > cf6d16b > > repository/src/test/java/org/apache/atlas/repository/migration/MigrationProgressServiceTest.java > 33125c8 > > repository/src/test/java/org/apache/atlas/repository/migration/TypesDefScrubberTest.java > d40ca82 > > repository/src/test/java/org/apache/atlas/repository/store/graph/AtlasTypeDefGraphStoreTest.java > b654638 > > repository/src/test/java/org/apache/atlas/repository/store/graph/v2/AtlasBusinessMetadataDefStoreV2Test.java > eaffac1 > > repository/src/test/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityStoreV2Test.java > b9cbef1 > > repository/src/test/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityTestBase.java > 752f1ac > > repository/src/test/java/org/apache/atlas/repository/store/graph/v2/AtlasRelationshipStoreV2Test.java > 24683a5 > > repository/src/test/java/org/apache/atlas/repository/store/graph/v2/BulkImportPercentTest.java > 1ae98ce > > repository/src/test/java/org/apache/atlas/repository/store/graph/v2/ClassificationAssociatorTest.java > d31b464 > > repository/src/test/java/org/apache/atlas/repository/store/graph/v2/InverseReferenceUpdateV2Test.java > 6364fd4 > > repository/src/test/java/org/apache/atlas/repository/tagpropagation/ClassificationPropagationTest.java > 80e20be > > repository/src/test/java/org/apache/atlas/repository/userprofile/UserProfileServiceTest.java > eeab3bc > > repository/src/test/java/org/apache/atlas/utils/ObjectUpdateSynchronizerTest.java > 03ebae4 > repository/src/test/java/org/apache/atlas/utils/TestLoadModelUtils.java > f175386 > > > Diff: https://reviews.apache.org/r/72566/diff/2/ > > > Testing > ------- > > Tried running all the Test Cases and they were running fine. > > > Thanks, > > mayank jain > >