address comments

Author: fresh-borzoni

Cargo.lock

@@ -54,10 +54,7 @@ defmodule Fluss.Native do
   def schema_builder_column(_builder, _name, _data_type), do: :erlang.nif_error(:nif_not_loaded)
   def schema_builder_primary_key(_builder, _keys), do: :erlang.nif_error(:nif_not_loaded)
   def schema_builder_build(_builder), do: :erlang.nif_error(:nif_not_loaded)
-  def table_descriptor_new(_schema), do: :erlang.nif_error(:nif_not_loaded)
-  def table_descriptor_with_bucket_count(_schema, _count), do: :erlang.nif_error(:nif_not_loaded)
-
-  def table_descriptor_with_properties(_schema, _properties),
+  def table_descriptor_new(_schema, _bucket_count, _properties),
     do: :erlang.nif_error(:nif_not_loaded)
 
   # Table

bindings/elixir/lib/fluss/native.ex

@@ -34,19 +34,10 @@ defmodule Fluss.TableDescriptor do
 
   @spec new!(Fluss.Schema.t(), keyword()) :: t()
   def new!(schema, opts \\ []) do
-    result =
-      cond do
-        Keyword.has_key?(opts, :bucket_count) ->
-          Native.table_descriptor_with_bucket_count(schema, opts[:bucket_count])
+    bucket_count = Keyword.get(opts, :bucket_count)
+    properties = Keyword.get(opts, :properties, [])
 
-        Keyword.has_key?(opts, :properties) ->
-          Native.table_descriptor_with_properties(schema, opts[:properties])
-
-        true ->
-          Native.table_descriptor_new(schema)
-      end
-
-    case result do
+    case Native.table_descriptor_new(schema, bucket_count, properties) do
       {:error, reason} -> raise "failed to create table descriptor: #{reason}"
       ref -> ref
     end