Skip to content

#59 Fix funnels not always saving and allow updating of name - #66

Merged
timgl merged 1 commit into
masterfrom
59-fix-funnel-names-not-always-saving
Feb 11, 2020
Merged

timgl merged 1 commit into
masterfrom
59-fix-funnel-names-not-always-saving

Conversation

@timgl

@timgl timgl commented Feb 11, 2020

Copy link
Copy Markdown
Collaborator

No description provided.

@timgl
timgl temporarily deployed to posthog-59-fix-funnel-n-h10rbo February 11, 2020 20:08 Inactive
@timgl
timgl merged commit 43dbbf8 into master Feb 11, 2020
@mariusandra
mariusandra deleted the 59-fix-funnel-names-not-always-saving branch April 30, 2020 19:53
yakkomajuri pushed a commit that referenced this pull request Oct 1, 2021
neilkakkar pushed a commit that referenced this pull request Jun 10, 2024
orian added a commit that referenced this pull request Jun 26, 2026
…x parsers

Replace the three text parsers (engine_map's `hclexp load` STDERR scrape,
parse_diff's statement/UNSAFE splitting, classify's op/table regex) and the
separate `hclexp load` call with a single `hclexp diff -format json` per node
(chschema #66). Each structured operation carries kind/object/sql/engine/
replicated/order/unsafe, so node_roles, sharded, is_alter_on_replicated_table,
and the CLOUD_DEPLOYMENT env-gating notes are derived from fields instead of
parsed from DDL text. Statements keep hclexp's per-node dependency order
(first-seen across the manifest) instead of an alphabetical sort. 226 -> 174 LOC.

This branch was previously deployed

1 inactive deployment
posthog-59-fix-funnel-n-h10rbo — 717b8d81 Deployed Feb 11, 2020 by timgl
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant