Home Original page

Remove NoSQL databases by malwaregarry · Pull Request #915 · sqlancer/sqlancer

@malwaregarry

Removed NoSQL databases so that it is easier to consolidate code for the refactoring effort.

Databases removed:

  • MongoDB
  • ArangoDB
  • Cosmos

@malwaregarry

mrigger

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! Just for documentation: these implementations are, to the best of my knowledge, not actively used and make it more difficult to maintain SQLancer and hinder the ongoing efforts to improve SQLancer's architecture.

albertZhangTJ pushed a commit to albertZhangTJ/sqlancer-lancerfuzz that referenced this pull request

Mar 20, 2024

@malwaregarry @albertZhangTJ

@NayameoW

Hi, thanks for your commit. I merged it into my working branch but encountered an issue. Here’s the exception that was thrown:
Exception in thread "main" java.util.ServiceConfigurationError: sqlancer.DatabaseProvider: Provider sqlancer.arangodb.ArangoDBProvider not found
Could you please help me with this? Thank you!

@mrigger

Did you try mvn clean? If that doesn't work, something might have gone wrong with your merge.

@NayameoW

Did you try mvn clean? If that doesn't work, something might have gone wrong with your merge.

Thanks! Something went wrong with my IDEA, and this really helps.