Skip to content

Comments

[cleanup][io] Remove Pulsar IO Twitter connector#25080

Merged
lhotari merged 3 commits intoapache:masterfrom
lhotari:lh-remove-pulsar-io-twitter
Dec 16, 2025
Merged

[cleanup][io] Remove Pulsar IO Twitter connector#25080
lhotari merged 3 commits intoapache:masterfrom
lhotari:lh-remove-pulsar-io-twitter

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented Dec 16, 2025

Motivation

The Twitter Firehose API no longer exists and the Pulsar IO Twitter connector has been outdated a long time ago.

Modifications

  • Remove Pulsar IO Twitter connector from the Pulsar code base
  • Replace usage of Twitter connector in tests with Datagen connector

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Documentation in apache/pulsar-site#1068

@lhotari lhotari added this to the 4.2.0 milestone Dec 16, 2025
@lhotari lhotari self-assigned this Dec 16, 2025
@github-actions github-actions bot added the doc-required Your PR changes impact docs and you will update later. label Dec 16, 2025
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.42%. Comparing base (3efc174) to head (042ee0f).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #25080       +/-   ##
=============================================
+ Coverage     38.49%   74.42%   +35.93%     
- Complexity    13275    34164    +20889     
=============================================
  Files          1864     1917       +53     
  Lines        146403   150462     +4059     
  Branches      17012    17491      +479     
=============================================
+ Hits          56354   111981    +55627     
+ Misses        82341    29552    -52789     
- Partials       7708     8929     +1221     
Flag Coverage Δ
inttests 26.40% <ø> (+0.01%) ⬆️
systests 22.86% <ø> (+0.09%) ⬆️
unittests 73.96% <ø> (+39.31%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...ain/java/org/apache/pulsar/io/core/PushSource.java 100.00% <ø> (+100.00%) ⬆️

... and 1425 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lhotari lhotari merged commit 82ab144 into apache:master Dec 16, 2025
89 of 91 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-required Your PR changes impact docs and you will update later. ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants