RD-15710: Store the table definitions so that they are reused when ta…

…bleDefinitions is called again (#37)

This upgrade of das-server permit to cache a DAS instance and reuse it
when recreating the same DAS. The code change is to store table
definitions once for all in a variable, so that if the same instance is
reused, it can immediately return its table definitions.

build.sbt

@@ -213,5 +213,5 @@ lazy val docker = (project in file("docker"))
   .settings(
     strictBuildSettings,
     dockerSettings,
-    libraryDependencies += "com.raw-labs" %% "das-server-scala" % "0.1.5" % "compile->compile;test->test"
+    libraryDependencies += "com.raw-labs" %% "das-server-scala" % "0.1.7" % "compile->compile;test->test"
   )

src/main/scala/com/rawlabs/das/salesforce/DASSalesforce.scala

@@ -101,7 +101,12 @@ class DASSalesforce(options: Map[String, String]) extends DASSdk with StrictLogg
 
   private val allTables = staticTables ++ dynamicTables ++ maybeDatedConversionRateTable
 
-  override def tableDefinitions: Seq[TableDefinition] = allTables.map(_.tableDefinition)
+  // These are the table definitions that will be returned to the client. It's stored
+  // in a val to avoid recalculating it every time it's accessed. We do that because we
+  // don't expect the tables to change during the lifetime of the DAS instance.
+  private val definitions = allTables.map(_.tableDefinition)
+
+  override def tableDefinitions: Seq[TableDefinition] = definitions
 
   override def functionDefinitions: Seq[FunctionDefinition] = Seq.empty