Add walk package to traverse an AST

[makenowjust]

Close #218

• Add ast-gen-walk
• Remove ColumnDefOptions and VectorIndexOption because they are unused.

[makenowjust]

@apstndb I merged your PR #264. Could you review this?

[apstndb]

Note: go.mod still have go 1.22.0 dilective so we can't use range over func in memefish development.

[makenowjust]

go directive in go.mod is also updated. Is it problematic?

[apstndb]

Yeah, seems like I had missed this.

[apstndb]

👍

[apstndb]

The known requirements have been addressed and there appear to be no issues that need to be fixed.

LGTM!

[apstndb]

Note:
We can update these lines to reflect current conventions.

memefish/ast/ast.go

Lines 40 to 46 in 0052764

	// Conventions:
	//
	// - Each node interface (except for Node) should have isXXX method (XXX must be a name of the interface itself).
	// - `isXXX` methods should be defined after the interface definition
	// and the receiver should be the non-pointer node struct type.
	// - Each node struct should have pos and end comments.
	// - Each node struct should have template lines in its doc comment.

[makenowjust]

@apstndb That's updated. Thank you!