Fixing path name of realistics scenarios and deleting files with dependencies #80

galilasmb · 2024-08-09T17:01:00Z

No description provided.

galilasmb · 2024-08-09T17:02:48Z

Os caminhos para os três cenários realistas estavam com (), gerando erro ao localizar o pacote com o padrão: org.pasta().main.method()

Removi os parênteses de pasta().

@barbosamaatheus isso influencia em algo no seu estudo? Se sim, deixa em uma branch separada, exclusivo para sdg.

barbosamaatheus · 2024-08-09T18:50:26Z

@barbosamaatheus isso influencia em algo no seu estudo? Se sim, deixa em uma branch separada, exclusivo para sdg.

Acredito que não @galilasmb

pauloborba · 2024-08-13T15:22:22Z

@barbosamaatheus isso influencia em algo no seu estudo? Se sim, deixa em uma branch separada, exclusivo para sdg.

Acredito que não @galilasmb

@barbosamaatheus se certificasse que continua tudo ok? @galilasmb isso pode afetar os experimentos de testes também? não entendi a causa dos renamings dos arquivos

galilasmb · 2024-09-30T16:25:33Z

Não faz sentido ter um nome de pasta com (), isso representa método, e o SDG tenta fazer esse padrão no input, pra não ter que ajustar nada no SDG, apenas renomear a pasta para um padrão correto que todos os outros cenários seguem.

galilasmb · 2024-10-08T14:07:31Z

Isso foi somente em três arquivos de realistic

pauloborba · 2024-10-08T14:40:37Z

@HACardoso podes verificar se há impacto no experimento de testes?

...s.CommonGramsTokenFilterFactoryTests/testQueryModeCommonGramsAnalysis()/right-left-lines.csv

...s.CommonGramsTokenFilterFactoryTests/testQueryModeCommonGramsAnalysis()/left-right-lines.csv

...ommongrams.CommonGramsTokenFilterFactoryTests/testCommonGramsAnalysis()/right-left-lines.csv

...ommongrams.CommonGramsTokenFilterFactoryTests/testCommonGramsAnalysis()/left-right-lines.csv

pauloborba · 2024-10-08T15:07:27Z

...4a915224264b4862b33f40cb8478c/original-without-dependencies/merge/tachyon-0.6.0-SNAPSHOT.jar

@galilasmb esse tachyon é uma biblioteca?

Eu não faço ideia, mas em uma das execuções, estava pegando esse arquivo e tinha dependência.

Não! segundo o readme do projeto: Tachyon comes in two parts: the server to serve images and the plugin to use it.

certo. então o código do sistema que estamos analisando usa tachyon como um servidor? se for isso, não é bem uma dependência externa no sentido que a gente estava falando, mas é algo a não ser considerado pelas análises mesmo

@galilasmb aqui

pauloborba · 2024-10-08T15:08:42Z

...412cae4aba1556eb88/original-without-dependencies/merge/dropwizard-example-0.7.1-SNAPSHOT.jar

@galilasmb aqui fiquei na dúvida, já que o jar sendo removido é também do dropwizard. não parece uma dependência externa. podes verificar o que tem nesse jar?

@barbosamaatheus fez essa avaliação dos .jars

Esse é o conteúdo do .jar

O que interessa para nossa analise seria apenas o pacote io

@barbosamaatheus estritamente, a análise deveria ser executada com todo o código do sistema que a gente está analisando, nesse caso dropwizard, não apenas com o pacote onde está a classe a ser analisada. pacotes como antlr e javax certamente são dependências externas e não deveriam ser considerados, mas precisamos saber se algum dos outros pacotes também faz parte do sistema dropwizard, ao invés de ser uma dependência externa, entendeu?

@galilasmb e @barbosamaatheus aqui

pauloborba · 2024-10-08T15:09:21Z

...a1556eb88/original-without-dependencies/merge/original-dropwizard-example-0.7.1-SNAPSHOT.jar

@galilasmb idem

Esse é o exemplo que o Matheus analisou que o .jar estava com dependência.

idem meu último comentário acima

Está sendo produzido vários arquivos duplicados, uns com dependências e outros sem. O arquivo correto do cenário de dropwizard não é esse, é o dropwizard-core-0.7.1-SNAPSHOT. E ao executar, está pegando o arquivo de outro módulo, example, quando o correto é o core.

pauloborba · 2024-10-08T15:09:58Z

...1b316cd1f5/original-without-dependencies/merge/druid-merger-0.2.8-SNAPSHOT-selfcontained.jar

@galilasmb aqui fiquei na dúvida, já que o jar sendo removido é também do druid. não parece uma dependência externa. podes verificar o que tem nesse jar?

Novamente nesse caso. a classe que queremos analisar está no pacote "com"

@barbosamaatheus estritamente, a análise deveria ser executada com todo o código do sistema que a gente está analisando, nesse caso druid, não apenas com o pacote onde está a classe a ser analisada. pacotes como scala e kafka certamente são dependências externas e não deveriam ser considerados, mas precisamos saber se algum dos outros pacotes também faz parte do sistema druid, ao invés de ser uma dependência externa, entendeu?

Como posso saber isso professor? o .jar que consideramos certo nesse exemplo é assim (druid-server-0.2.8-SNAPSHOT.jar):

pelo nome do pacote, vendo se tem o mesmo padrão/nome da organização

aqui @galilasmb

pauloborba · 2024-10-08T15:10:11Z

...316cd1f5/original-without-dependencies/merge/druid-services-0.2.8-SNAPSHOT-selfcontained.jar

@galilasmb aqui fiquei na dúvida, já que o jar sendo removido é também do druid. não parece uma dependência externa. podes verificar o que tem nesse jar?

@galilasmb ?

Aqui o mesmo se aplica, esse não é o arquivo correto, está pegando o módulo services, mas o correto é server, conforme o jar correto: druid-server-0.2.8-SNAPSHOT.jar

barbosamaatheus

@galilasmb no lugar de apagar, não seria ideal guardar esses .jars em um local diferente para caso seja necessário no futuro?

barbosamaatheus · 2024-10-08T16:31:43Z

...4a915224264b4862b33f40cb8478c/original-without-dependencies/merge/tachyon-0.6.0-SNAPSHOT.jar

Não! segundo o readme do projeto: Tachyon comes in two parts: the server to serve images and the plugin to use it.

barbosamaatheus · 2024-10-08T16:34:59Z

...412cae4aba1556eb88/original-without-dependencies/merge/dropwizard-example-0.7.1-SNAPSHOT.jar

Esse é o conteúdo do .jar

O que interessa para nossa analise seria apenas o pacote io

barbosamaatheus · 2024-10-08T16:39:18Z

...1b316cd1f5/original-without-dependencies/merge/druid-merger-0.2.8-SNAPSHOT-selfcontained.jar

Novamente nesse caso. a classe que queremos analisar está no pacote "com"

HACardoso · 2024-10-09T13:41:34Z

@HACardoso podes verificar se há impacto no experimento de testes?

@pauloborba não detectei nenhum impacto nos respectivos experimentos com testes

This reverts commit 61b0e88.

barbosamaatheus

Aprovando, pois acredito que esse PR não causa nenhum prejuízo para as análises ou mergedetaset. Os jar apenas foram movidos e a remoção dos () também não chega a influenciar negativamente.

Entendo que ainda existe um ponto em discussão para os casos de .jar com/sem dependências

galilasmb · 2024-10-16T13:44:37Z

Falta algo mais para que esse PR seja integrado? @pauloborba @barbosamaatheus

pauloborba · 2024-10-16T14:04:40Z

Falta algo mais para que esse PR seja integrado? @pauloborba @barbosamaatheus

vê as pendências que marquei acima @galilasmb

fixing path name of realistics scenarios

33653ba

galilasmb requested review from pauloborba and barbosamaatheus August 9, 2024 17:02

barbosamaatheus approved these changes Aug 9, 2024

View reviewed changes

galilasmb requested a review from barbosamaatheus September 30, 2024 16:23

deleting files with dependencies

61b0e88

galilasmb changed the title ~~Fixing path name of realistics scenarios~~ Fixing path name of realistics scenarios and deleting files with dependencies Oct 8, 2024

pauloborba requested a review from HACardoso October 8, 2024 14:40

pauloborba requested changes Oct 8, 2024

View reviewed changes

barbosamaatheus reviewed Oct 8, 2024

View reviewed changes

HACardoso approved these changes Oct 9, 2024

View reviewed changes

galilasmb added 4 commits October 10, 2024 17:40

Revert "deleting files with dependencies"

64588a3

This reverts commit 61b0e88.

moving files to extras

9c979fa

removing ( )

9f2b150

renaming folter

6f47c87

galilasmb requested review from barbosamaatheus and pauloborba October 10, 2024 20:47

barbosamaatheus approved these changes Oct 10, 2024

View reviewed changes

Fixing path name of realistics scenarios and deleting files with dependencies #80

Are you sure you want to change the base?

Fixing path name of realistics scenarios and deleting files with dependencies #80

Conversation

galilasmb commented Aug 9, 2024

galilasmb commented Aug 9, 2024 • edited Loading

barbosamaatheus commented Aug 9, 2024

pauloborba commented Aug 13, 2024

galilasmb commented Sep 30, 2024

galilasmb commented Oct 8, 2024

pauloborba commented Oct 8, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

barbosamaatheus left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

HACardoso commented Oct 9, 2024

barbosamaatheus left a comment

Choose a reason for hiding this comment

galilasmb commented Oct 16, 2024

pauloborba commented Oct 16, 2024

galilasmb commented Aug 9, 2024 •

edited

Loading