update txservice subm

[yi-xmu]

Summary by CodeRabbit

• Chores
  • Updated an internal dependency reference; no functional changes were introduced.
  • Low-impact maintenance to keep internal components synchronized and up to date.
  • This update requires no user action and does not affect existing functionality, performance, or user-facing behavior.

[coderabbitai]

Walkthrough

The storage/eloq/tx_service submodule reference is updated from commit b5f974df8a9d66dcb29436376105d59adc6512da to b0081d751ce36801a346c1d749c99e72742c237b. No functional code changes detected.

Changes

Cohort / File(s)	Change Summary
Submodule Update storage/eloq/tx_service	Submodule commit reference changed from b5f974df8a9d66dcb29436376105d59adc6512da to b0081d751ce36801a346c1d749c99e72742c237b (metadata-only).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Areas to spot-check:

• Ensure only the submodule pointer changed (no staged code edits).
• Confirm submodule URL/path remains correct.

Possibly related PRs

• update submodules #149: Updates the same storage/eloq/tx_service submodule pointer (similar commit-bump).
• update subm #117: Another PR that adjusts the tracked commit for storage/eloq/tx_service.
• Update submodule for txservice #161: Advances the storage/eloq/tx_service pointer in adjacent history (related pointer movement).

Suggested reviewers

• liunyl
• lzxddz

Poem

🐰 I nudged a hash, a tiny hop,
From one commit to the next I pop.
No code disturbed, just metadata spun,
A quiet hop — the update’s done. ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'update txservice subm' is incomplete and uses unclear abbreviation ('subm'). It lacks clarity about what is being updated and why.	Expand the title to be complete and descriptive, such as 'Update txservice submodule commit reference' or 'Bump txservice submodule to latest version'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch update_txservice_subm

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ff8c3a0 and 1316dfb.

📒 Files selected for processing (1)

• storage/eloq/tx_service (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• storage/eloq/tx_service

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 92d2110 and ff8c3a0.

📒 Files selected for processing (1)

• storage/eloq/tx_service (1 hunks)

[coderabbitai]

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

Verify the submodule changes and test integration.

This updates the tx_service submodule reference. Please ensure that the changes in the submodule have been reviewed and that the integration has been tested with the parent repository.

Run the following script to inspect what changed in the submodule:

---

🏁 Script executed:

#!/bin/bash
# Description: Review the submodule commit history and changes

cd storage/eloq/tx_service

# Verify the new commit exists
echo "=== Verifying new commit exists ==="
git cat-file -t 2ace73c919ebbdb0c3823dfd89494d84e74a467b 2>/dev/null || echo "Commit not found in submodule"

# Show commit history between old and new references
echo -e "\n=== Commits included in this update ==="
git log --oneline b5f974df8a9d66dcb29436376105d59adc6512da..2ace73c919ebbdb0c3823dfd89494d84e74a467b

# Show summary of changes
echo -e "\n=== Summary of file changes ==="
git diff --stat b5f974df8a9d66dcb29436376105d59adc6512da..2ace73c919ebbdb0c3823dfd89494d84e74a467b

# Show detailed changes (limit output)
echo -e "\n=== Detailed changes (first 100 lines) ==="
git diff b5f974df8a9d66dcb29436376105d59adc6512da..2ace73c919ebbdb0c3823dfd89494d84e74a467b | head -100

Length of output: 699

---

🏁 Script executed:

#!/bin/bash
# Description: Diagnose submodule configuration and remote issues

cd storage/eloq/tx_service

echo "=== Submodule Configuration ==="
git config -l | grep -E '(url|remote)'

echo -e "\n=== Remote URL ==="
git remote -v

echo -e "\n=== Submodule Remote Info ==="
git ls-remote --refs origin 2>&1 | head -20

echo -e "\n=== Current HEAD and Branch ==="
git rev-parse HEAD
git branch -a

echo -e "\n=== Check if commits are reachable from any ref ==="
git for-each-ref --format='%(refname) %(objectname)' | head -20

echo -e "\n=== Submodule .git info ==="
if [ -d .git ]; then
  echo "Submodule has .git directory"
  ls -la .git | head -5
elif [ -f .git ]; then
  echo "Submodule has .git file (worktree/shallow):"
  cat .git
fi

Length of output: 2320

---

🏁 Script executed:

#!/bin/bash
# Description: Search for target commits across all remote branches

cd storage/eloq/tx_service

echo "=== Checking if target commits exist on any branch ==="
TARGET_OLD="b5f974df8a9d66dcb29436376105d59adc6512da"
TARGET_NEW="2ace73c919ebbdb0c3823dfd89494d84e74a467b"

echo "Searching for old commit: $TARGET_OLD"
git branch -r --contains $TARGET_OLD 2>&1 | head -5

echo -e "\nSearching for new commit: $TARGET_NEW"
git branch -r --contains $TARGET_NEW 2>&1 | head -5

echo -e "\n=== Attempting fetch from remote to populate all refs ==="
git fetch origin --dry-run 2>&1 | head -20

echo -e "\n=== Checking log on current coderabbit_160 branch ==="
git log --oneline -10

echo -e "\n=== Check if commits might be on main branch ==="
git show b5f974df8a9d66dcb29436376105d59adc6512da 2>&1 | head -5
git show 2ace73c919ebbdb0c3823dfd89494d84e74a467b 2>&1 | head -5

Length of output: 1589

---

Submodule references point to non-existent commits—update cannot proceed.

The PR updates the tx_service submodule to commits that do not exist in the remote repository:

• Old reference b5f974df8a9d66dcb29436376105d59adc6512da: Not found
• New reference 2ace73c919ebbdb0c3823dfd89494d84e74a467b: Not found

Both commits return "not our ref" errors from https://github.com/eloqdata/eloqsql.git, meaning they are unreachable from any branch or tag. This will cause checkout failures in CI/CD and deployment.

Verify the correct commit SHAs for the submodule update, or ensure the commits have been properly pushed to the remote repository before proceeding.

🤖 Prompt for AI Agents

In storage/eloq/tx_service around lines 1 to 1, the submodule update points to
commit SHAs that are not reachable in the remote repo causing "not our ref"
errors; verify and correct the submodule commit references by checking the
remote (https://github.com/eloqdata/eloqsql.git) for the intended commit(s) or
branch/tag, update .gitmodules/.git/config and the submodule entry to a valid
pushed SHA or a branch ref, or push the missing commits to the remote so the
referenced SHAs become reachable, then update the submodule pointer and commit
the corrected submodule state.

[yi-xmu]

Updated by pr:#164