objectbox / objectbox-java

Android Database - first and fast, lightweight on-device vector database
https://objectbox.io
Apache License 2.0
4.39k stars 302 forks source link

Support Gradle build cache #1114

Open rAseri opened 1 year ago

rAseri commented 1 year ago

The bug is occurred in an Android Project.

Description of the bug During a manual conflict resolution, the objectbox-models/default.json file may accidentally contain duplicated indexId entries. During the build time, we don't have any errors. The project builds successfully, and the build output is cached to the Gradle Build Cache. Only at runtime, when we build BoxStore instance, an error occurred: io.objectbox.exception.DbSchemaException: Duplicate index ID 2 found for property .... Moreover, when a developer fixes duplicated indexId entries in the objectbox-models/default.json file and rebuilds the project, the error is still in place due to the result of the previous build is used from the Gradle Build Cache. It incurs hard-to-debug problems for our dev team.

Basic info:

Steps to reproduce the behavior:

  1. Activate the Gradle Build Cache: add org.gradle.caching=true in the gradle.properties file.
  2. Add two ObjectBox entities (e.g. User and Company) with the unique name property.
  3. Build the project to generate a objectbox-models/default.json file.
  4. Make changes to the objectbox-models/default.json file: set the same value to both indexId entries (Company.name and User.name).
  5. Rebuild the project.
  6. Install APK and launch the app. There is the error: io.objectbox.exception.DbSchemaException: Duplicate index ID 2 found for property ...
  7. Fix duplicated indexId in the objectbox-models/default.json file.
  8. Rebuild the project.
  9. Install APK and launch the app. There is still the same error. The result of the previous build is used.

Expected behavior

Code I've prepared the demo project to reproduce the problem with the described above steps: https://github.com/rAseri/ObjectBoxDemo

Logs, stack traces

Process: com.example.myapplication, PID: 24377
java.lang.RuntimeException: Unable to start activity ComponentInfo{com.example.myapplication/com.example.myapplication.MainActivity}: io.objectbox.exception.DbSchemaException: Duplicate index ID 2 found for property User.name
    at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3449)
    at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3601)
    at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:85)
    at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135)
    at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:95)
    at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2066)
    at android.os.Handler.dispatchMessage(Handler.java:106)
    at android.os.Looper.loop(Looper.java:223)
    at android.app.ActivityThread.main(ActivityThread.java:7656)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)
greenrobot-team commented 1 year ago

Thanks for reporting! I guess the culprit is that the model file is only touched by the annotation processor. We need to check if it can somehow be indicated to Gradle that the processor has the model file as an input and output. ~Also likely the generated code as output (or is it already?).~ The generated code is created using the Filer API, where output is monitored by Gradle.

A workaround could be to add the model file as an input to e.g. an assemble task (or whichever necessary):

tasks.named("assembleDebug") {
    inputs.property("objectboxModel") {
        file("objectbox-models/default.json")
    }
}

Maybe we can also do this automatically with the Gradle plugin.

(Somewhat related, though configuration cache other than build cache is still experimental: https://github.com/objectbox/objectbox-java/issues/948)

rAseri commented 1 year ago

Thanks for your reply!

I am not a Gradle expert, but it looks like a working solution (I'm using Kotlin Scripts here):

    plugins {
        id("io.objectbox")
        // ...
    }

    // ...
    tasks.withType<org.jetbrains.kotlin.gradle.tasks.BaseKapt>().configureEach {
        inputs.file("objectbox-models/default.json")
    }

I added the model file as an input for BaseKapt task. Now every change in the model file causes the kaptDebugKotlin task to rerun as expected! Screenshot 2023-02-13 at 17 55 39

rAseri commented 1 year ago

Maybe we can also do this automatically with the Gradle plugin.

I think it's a good idea!

greenrobot-team commented 1 year ago

@rAseri Thanks for testing this! Looks like adding an input to the kapt (or Java equivalent annotation processor) task is what we will let the Gradle plugin do then.

(Added an internal issue for this.)