Skip to content
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

Legacy Data failing to appear in sponge.data-registrations.failed-data-list #1714

Closed
ryantheleach opened this issue Jan 15, 2018 · 5 comments
Assignees
Labels
system: data type: bug Something isn't working version: 1.12 (u) API: 7 (unsupported since May 21st 2021)
Milestone

Comments

@ryantheleach
Copy link
Contributor

ryantheleach commented Jan 15, 2018

@Big-Deaddy is currently running

  • SpongeForge version: Using SF 2833
  • SpongeVanilla version: Untested
  • Java version: Unknown
  • Operating System: Unknown

Issue Description
It's been reported in IRC that removing https://github.com/hsyyid/CommandSigns/tree/master/src/main/java/io/github/hsyyid/commandsigns results in spam.

image

But fails to populate in the failed data configuration: https://gist.github.com/Big-Deaddy/2e20c4ddded6fa8efbdfd42250162eba

It's my suspicion that this is due to the legacy data not being populated in the list, or even having a method listed in the config of the specification to list it.

@ryantheleach ryantheleach added type: bug Something isn't working system: data version: 1.12 (u) API: 7 (unsupported since May 21st 2021) labels Jan 15, 2018
@ryantheleach ryantheleach added this to the API 7.1 milestone Jan 15, 2018
@gabizou gabizou added this to Working on a Fix in gabizou's TODO List Jan 15, 2018
@gabizou gabizou moved this from Working on a Fix to Acknowledged in gabizou's TODO List Jan 15, 2018
@KyleFreedomSeeds
Copy link

bump. Im experiencing the same bug, The spam is so massive it lags out console and freezes the sever for around 30 seconds. Hoping for a fix :D

@gabizou
Copy link
Member

gabizou commented Feb 9, 2018

Once the build finishes, can you try build 2892, it should very well accurately save the config as failed data is discovered.

@johnfriedrich
Copy link

This is fixed, however entries put in data-to-purge won't get purged from disk. At least is that, what I expect this option should do

@gabizou
Copy link
Member

gabizou commented Feb 21, 2018

It should be purging the data as soon as it is loaded from chunks/entities/etc. Could you provide an example of NBT data for me to double check and test?

@dualspiral
Copy link
Contributor

No response, closing.

gabizou's TODO List automation moved this from Acknowledged to Done Jun 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
system: data type: bug Something isn't working version: 1.12 (u) API: 7 (unsupported since May 21st 2021)
Projects
Development

No branches or pull requests

5 participants