fix: Add Cython build step to Makefile

[geruh]

Rationale for this change

Currently, uv sync is building the Cython extensions on first install. After running make clean (which deletes .so files), subsequent uv sync does not rebuild them, causing:

  ModuleNotFoundError: No module named 'pyiceberg.avro.decoder_fast'

This seems to be related to some known uv behavior: astral-sh/uv#12399. The --reinstall flag forces uv to rebuild the package and its extensions.

Note: CI works because it always starts fresh. Local dev breaks after make clean.

Are these changes tested?

make clean && make test

Are there any user-facing changes?

new make target

[kevinjqliu]

TY! Interesting uv behavior.

I tested this locally with

make clean && make install && PYTEST_ARGS='-k tests/avro/test_decoder.py' make test

this fails on current main because the cython extension is not built, but passes here

[kevinjqliu]

the cython extension fails silently in dev setup

iceberg-python/setup.py

Lines 39 to 72 in b0a7878

	allowed_to_fail = os.environ.get("CIBUILDWHEEL", "0") != "1"
	ext_modules = []
	try:
	import Cython.Compiler.Options
	from Cython.Build import cythonize
	Cython.Compiler.Options.annotate = True
	if os.name == "nt": # Windows
	extra_compile_args = ["/O2"]
	else: # UNIX-based systems (Linux, macOS)
	extra_compile_args = ["-O3"]
	package_path = "pyiceberg"
	extensions = [
	Extension(
	"pyiceberg.avro.decoder_fast",
	[os.path.join(package_path, "avro", "decoder_fast.pyx")],
	extra_compile_args=extra_compile_args,
	language="c",
	)
	]
	ext_modules = cythonize(
	extensions,
	include_path=[package_path],
	compiler_directives={"language_level": "3"},
	annotate=True,
	)
	except Exception:
	if not allowed_to_fail:

but will throw when fail during artifact build.

iceberg-python/.github/workflows/pypi-build-artifacts.yml

Lines 63 to 78 in b0a7878

	- name: Build wheels
	uses: pypa/cibuildwheel@v3.3.0
	with:
	output-dir: wheelhouse
	config-file: "pyproject.toml"
	env:
	# Ignore 32 bit architectures
	CIBW_ARCHS: "auto64"
	CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10"
	CIBW_TEST_REQUIRES: "pytest==7.4.2 moto==5.0.1"
	CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py"
	# Ignore tests for pypy since not all dependencies are compiled for it
	# and would require a local rust build chain
	CIBW_TEST_SKIP: "pp*"
	# Skip free-threaded (PEP 703) builds until we evaluate decoder_fast support
	CIBW_SKIP: "cp3*t-*"